Skip to content

Provide a way to manually exclude some template prompts/notifications#178

Merged
ukmo-ccbunney merged 4 commits intoSciTools:mainfrom
trexfeathers:template_quiet
Jul 31, 2025
Merged

Provide a way to manually exclude some template prompts/notifications#178
ukmo-ccbunney merged 4 commits intoSciTools:mainfrom
trexfeathers:template_quiet

Conversation

@trexfeathers
Copy link
Contributor

@trexfeathers trexfeathers commented Jul 11, 2025

I haven't done much diligence/testing, just wanted to get the idea out there before I forgot it.

@trexfeathers
Copy link
Contributor Author

notify-updates text example

The template for LICENSE has been updated; see the diff below. Please either:

  • Action this issue with a pull request applying some/all of these changes to LICENSE. 1
  • Close this issue if none of these changes are appropriate for this repo.

Also consider reviewing a full diff between the template and LICENSE, in case other valuable shared conventions have previously been missed.

File Links

Diff

Footnotes

  1. Include this text in the PR body to avoid any prompts about applying your changes back to the template!
    @scitools-templating: please no share prompt

@trexfeathers
Copy link
Contributor Author

prompt-share text example

Templating

SciTools/foo#1 (by @trexfeathers) includes changes that may be worth sharing via templating. For each file listed below, please either:

  • Action the suggestion via a pull request editing/adding the relevant file in the templates directory. 1
  • Dismiss the suggestion if the changes are not suitable for templating.

Templated files

The following changed files are templated:

Footnotes

  1. Include this text in the PR body to avoid any notifications about applying the template changes back to the source repo!
    @scitools-templating: please no update notification on: iris

Copy link
Contributor

@ukmo-ccbunney ukmo-ccbunney left a comment

Choose a reason for hiding this comment

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

I've tested the notify-updates locally on my personal fork.
All looks good apart from one necessary change.

@ukmo-ccbunney
Copy link
Contributor

Finished testing the prompt-share changes now. This is working fine and looking good.

Co-authored-by: Chris Bunney <48915820+ukmo-ccbunney@users.noreply.github.com>
@trexfeathers trexfeathers marked this pull request as ready for review July 31, 2025 16:13
Copy link
Contributor

@ukmo-ccbunney ukmo-ccbunney left a comment

Choose a reason for hiding this comment

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

Great stuff! I think this is good to go! 🚀

@ukmo-ccbunney ukmo-ccbunney merged commit 859fc11 into SciTools:main Jul 31, 2025
2 checks passed
@trexfeathers
Copy link
Contributor Author

Thanks @ukmo-ccbunney!

@scitools-ci scitools-ci bot removed this from 🚴 Peloton Aug 29, 2025
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