Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add .html to archive file names without extension #127

Closed
wants to merge 1 commit into from

Conversation

Exagone313
Copy link

Fixes #102.

Read carefully before merging, this may break some configurations, I need your feedback.
Also this is the first time I'm writing Ruby.

@Exagone313
Copy link
Author

Exagone313 commented Jun 3, 2018

Hmm, the commit message may not be good as I'm checking for \.html$ and not if there isn't an extension.

@ashmaroli
Copy link
Member

@Exagone313 Are you still interested in getting this merged..?

@Exagone313
Copy link
Author

It requires some changes but I still want to fix the underlying issue I created.

@ashmaroli
Copy link
Member

Okay. Apart from amending the original commit message (git commit --amend), you may want to check the source code for Jekyll::Page#destination and Jekyll::Document#destination for logic reference.

@Exagone313
Copy link
Author

I don't have time to work on this PR, I'm closing it.

@Exagone313 Exagone313 closed this Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically add .html to generated archives names without extension
2 participants