Thanks to visit codestin.com
Credit goes to github.com

Skip to content

+ [Feature] add the ignore hidden files option to publish #53

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
Merged
merged 2 commits into from
Dec 2, 2016
Merged

+ [Feature] add the ignore hidden files option to publish #53

merged 2 commits into from
Dec 2, 2016

Conversation

snowyu
Copy link
Contributor

@snowyu snowyu commented Nov 11, 2016

the github pages need the .nojekyll file to work now.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 87.387% when pulling e55a89f on snowyu:feature/ignoreHidden into 6bac498 on hexojs:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 87.387% when pulling e55a89f on snowyu:feature/ignoreHidden into 6bac498 on hexojs:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 87.387% when pulling e55a89f on snowyu:feature/ignoreHidden into 6bac498 on hexojs:master.

@NoahDragon
Copy link
Member

NoahDragon commented Nov 11, 2016 •
edited
Loading

Your solution is more generic, close #52

Could you please fix the eslint errors and update the test cases?

@NoahDragon NoahDragon mentioned this pull request Nov 11, 2016
Closed
@coveralls
Copy link

coveralls commented Nov 13, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.8%) to 87.387% when pulling 45eaedc on snowyu:feature/ignoreHidden into 6bac498 on hexojs:master.

@snowyu
Copy link
Contributor Author

snowyu commented Nov 13, 2016

eslint done. But I have no idea and time about testing. sorry. The logical is simple and clear.

@NoahDragon
Copy link
Member

NoahDragon commented Dec 2, 2016 •
edited
Loading

I will add the test for this.

Test added #54.

@NoahDragon NoahDragon merged commit fa14901 into hexojs:master Dec 2, 2016
@NoahDragon NoahDragon mentioned this pull request Dec 2, 2016
Merged
NoahDragon referenced this pull request Dec 2, 2016
@NoahDragon
Merge pull request #54 from hexojs/Add-test-for-#53
924a843 
Update deployer.js
NoahDragon added a commit that referenced this pull request Dec 2, 2016
@NoahDragon
Add test case for #53
5f6284c
@coveralls coveralls mentioned this pull request Dec 2, 2016
Merged
@NoahDragon NoahDragon mentioned this pull request Dec 2, 2016
Closed
NoahDragon added a commit that referenced this pull request Dec 2, 2016
@NoahDragon
Add test case for #53 (#55)
83fc0df 
* Add test case for #53

* Update deployer.js

* Update deployer.js

* Update deployer.js

* Update deployer.js
@NoahDragon NoahDragon mentioned this pull request Dec 5, 2016
Closed
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
None yet
Projects
None yet
Milestone
No milestone
Development

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