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

[Template Sync] Drift from template-repository remediation #120

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 1, 2024

Resultant drift from repository template

Who should respond to this PR?

The team which owns the responsibility for this component repository. You may want to consult other contributors.

How to progress this PR

The repositories guardians should review the contents of the PR and decide how to proceed, you may wish to back-out certain changes or accept them from the upstream nhsdigital/nhs-notify-repository-template repository.

If there are changes you do not wish to see again, it is recommended you add exclusions to scripts/config/.repository-template-sync-ignore.

@github-actions github-actions bot requested review from a team as code owners October 1, 2024 00:26
@github-actions github-actions bot requested a review from a team as a code owner October 1, 2024 00:26
group_target-env.tfvars

.amplify/
amplify_outputs.json
Copy link
Contributor

Choose a reason for hiding this comment

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

This must not be removed from the gitignore.

Copy link
Contributor

Choose a reason for hiding this comment

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

Files can be added to scripts/config/.repository-template-sync-ignore


@app.route("/")
def index():
return "Hello World!"
Copy link
Contributor

Choose a reason for hiding this comment

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

Lets not add unnecessary code into the repo.

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree Files can be added to scripts/config/.repository-template-sync-ignore

Copy link
Contributor

@mark-r-bjss mark-r-bjss left a comment

Choose a reason for hiding this comment

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

This should not be merged trivially. The template contains lots of "examples" that do not necessarily make sense to be in every repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants