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

feat($core): config pattern of resolved files #1705

Merged
merged 4 commits into from
Oct 8, 2019

Conversation

eduardcotmrf
Copy link
Contributor

Summary

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • [x ] Feature
  • Code style update
  • Refactor
  • Docs
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • [x ] No

The PR fulfills these requirements:

Right now the pattern configuration of what files should be resolved as pages is hardcoded in core module of vuepress (APP.js line 319). The intention it's to open the door to parametrizing this pattern configuration from the siteConfig.

fix #1700

You have tested in the following browsers: (Providing a detailed version will be better.)

  • [ x] Chrome
  • [ x] Firefox
  • [x ] Safari
  • Edge
  • IE

If adding a new feature, the PR's description includes:

  • [ x] A convincing reason for adding this feature
  • [ x] Related documents have been updated
  • [ x] Related tests have been updated

@flozero
Copy link
Collaborator

flozero commented Sep 5, 2019

Hello @eduardcotmrf Thank's for the catch ! Can you just add me a real example that it can be great. And it is perfect ! 👍

@flozero flozero added status: community assigned Community assigned topic: config Relates to VuePress config type: enhancement Request to enhance an existing feature version: 1.x Relates to version 1 of VuePress labels Sep 5, 2019
Copy link
Collaborator

@kefranabg kefranabg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eddyerburgh Thanks for this PR !

@f3ltron a scenario is described here

@flozero
Copy link
Collaborator

flozero commented Sep 20, 2019

can you add into the zh doc too please even if it's in english

@eduardcotmrf
Copy link
Contributor Author

Hey, i just came back from paternity leave, sorry for not answering for some time :)
Here i added the doc to zh as well in english.
Thanks!

@eduardcotmrf eduardcotmrf removed their assignment Oct 8, 2019
@eduardcotmrf
Copy link
Contributor Author

eduardcotmrf commented Oct 8, 2019

Can you merge it @f3ltron ?
Thanks!

@flozero flozero merged commit 1f3e4e2 into vuejs:master Oct 8, 2019
@vue-bot
Copy link

vue-bot commented Oct 8, 2019

Hey @eduardcotmrf, thank you for your time and effort spent on this PR, contributions like yours help make Vue better for everyone. Cheers! 💚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: community assigned Community assigned topic: config Relates to VuePress config type: enhancement Request to enhance an existing feature version: 1.x Relates to version 1 of VuePress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parametrize pattern of pages resolved
4 participants