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 smv_prebuild_export_pattern and smv_prebuild_export_destination options #1

Open
wants to merge 3 commits into
base: prebuild_command
Choose a base branch
from

Conversation

samuel-emrys
Copy link

  • smv_prebuild_export_pattern will allow a user to specify a regular expression to identify files and folders resulting from smv_prebuild_command to the versioned output directory
  • smv_prebuild_export_destination will allow the user to specify the name of a directory in the versioned output directory in which to place the files matched by smv_prebuild_export_pattern.
  • smv_postbuild_export_pattern and smv_postbuild_export_destination also added with the same functionality as above, except applied to the results of smv_postbuild_command
  • Add descriptions of these to the documentation, with exemplar usage.

samtygier-stfc and others added 3 commits May 26, 2021 20:08
Can be used to run a command in the checked out director before building
with sphinx.
…ptions

* smv_prebuild_export_pattern will allow a user to specify a regular
  expression to identify files and folders resulting from
  smv_prebuild_command to the versioned output directory
* smv_prebuild_export_destination will allow the user to specify the
  name of a directory in the versioned output directory in which to
  place the files matched by smv_prebuild_export_pattern.
* smv_postbuild_export_pattern and smv_postbuild_export_destination also
  added with the same functionality as above, except applied to the
  results of smv_postbuild_command
* Add descriptions of these to the documentation, with exemplar usage.
@samuel-emrys
Copy link
Author

samuel-emrys commented Jan 22, 2022

Hi @samtygier-stfc, hope you don't mind, I've made some changes as per my comment at sphinx-contrib#62 (comment) and added a PR to your repo so that, should you accept, they'll be included in sphinx-contrib#62. Happy to accept any comments/feedback if you have any.

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.

2 participants