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

Upgrade the global gitignore files with some more files to ignore #18317

Merged
merged 2 commits into from
Apr 3, 2024

Conversation

saiarcot895
Copy link
Contributor

Why I did it

Specifically, ignore the Dockerfile.cleanup files that are autogenerated for the docker containers, and ignore the udeb files built as part of openssh.

There are a lot more files that should be ignored, but this is taking care of the easy ones.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Specifically, ignore the Dockerfile.cleanup files that are autogenerated
for the docker containers, and ignore the udeb files built as part of
openssh.

There are a lot more files that should be ignored, but this is taking
care of the easy ones.

Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
@k-v1
Copy link
Contributor

k-v1 commented Mar 12, 2024

Probably also need add to ignore

platform/*/docker-*/Dockerfile-dbg.cleanup
dockers/*/Dockerfile-dbg.cleanup

Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
@yxieca yxieca merged commit 2a63b61 into sonic-net:master Apr 3, 2024
19 checks passed
@saiarcot895 saiarcot895 deleted the update-gitignore branch April 9, 2024 15:58
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.

3 participants