Skip to content

Add smv_prebuild_export_pattern and smv_prebuild_export_destination options#1

Open
samuel-emrys wants to merge 3 commits into
samtygier-stfc:prebuild_commandfrom
samuel-emrys:prebuild_command
Open

Add smv_prebuild_export_pattern and smv_prebuild_export_destination options#1
samuel-emrys wants to merge 3 commits into
samtygier-stfc:prebuild_commandfrom
samuel-emrys:prebuild_command

Conversation

@samuel-emrys

Copy link
Copy Markdown
  • 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

samuel-emrys commented Jan 22, 2022

Copy link
Copy Markdown
Author

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