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.

Sign up for GitHub

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

Jump to bottom

Fix the #2134 #2551

Closed
wants to merge 1 commit into from
Closed

Fix the #2134 #2551

wants to merge 1 commit into from

Conversation

Alex1990
Copy link

@Alex1990 Alex1990 commented May 4, 2017

Thank you for creating a pull request to contribute to Hexo code! Before you open the request please review the following guidelines and tips to help it be more easily integrated:

  • Add test cases for the changes.
  • Passed the CI test.

@coveralls
Copy link

coveralls commented May 4, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.02%) to 97.165% when pulling f9ee66e on Alex1990:master into bbc643a on hexojs:master.

@stale stale bot added the wontfix This will not be worked on label Sep 27, 2017
@hexojs hexojs deleted a comment from stale bot Sep 27, 2017
@NoahDragon NoahDragon added enhancement New feature or request and removed wontfix This will not be worked on labels Sep 27, 2017
@NoahDragon
Copy link
Member

@Alex1990 Could you please add the test case?

JLHwung added a commit that referenced this pull request Dec 4, 2017
@JLHwung
fix(post-asset): strip /\.html?$/ extensions on permalink
971c56e 
Fixes #2134
Closes #2551
Closes #2504
@JLHwung JLHwung mentioned this pull request Dec 4, 2017
Merged
2 tasks
JLHwung added a commit that referenced this pull request Dec 4, 2017
@JLHwung
fix(post-asset): strip /\.html?$/ extensions on permalink
597bf1b 
Fixes #2134
Closes #2551
Closes #2504
JLHwung added a commit that referenced this pull request Dec 4, 2017
@JLHwung
fix(post-asset): strip /\.html?$/ extensions on permalink
91d6110 
Fixes #2134
Closes #2551
Closes #2504
NoahDragon pushed a commit that referenced this pull request Dec 4, 2017
@JLHwung @NoahDragon
fix(post-asset): strip /\.html?$/ extensions on permalink (#2881)
284c1b9 
Fixes #2134
Closes #2551
Closes #2504
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Alex1990 @coveralls @NoahDragon