Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Do not mark package removals from attw.json as "Edits Infrastructure" #487

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

andrewbranch
Copy link
Member

No description provided.

@@ -0,0 +1,17 @@
{
"projectColumn": "Waiting for Code Reviews",
"labels": [],
Copy link
Member Author

Choose a reason for hiding this comment

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

This was a package removal

{
"projectColumn": "Waiting for Code Reviews",
"labels": [
"Edits Infrastructure"
Copy link
Member Author

Choose a reason for hiding this comment

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

This was a package name edit (i.e. one removal one insertion)

{
"projectColumn": "Waiting for Code Reviews",
"labels": [
"Edits Infrastructure"
Copy link
Member Author

Choose a reason for hiding this comment

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

This was a removal of a config line outside of failingPackages

Copy link
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

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

Seems correct to me.

@andrewbranch andrewbranch merged commit e2955e6 into master Feb 7, 2024
2 checks passed
@andrewbranch andrewbranch deleted the allow-safe-attw-removals branch February 7, 2024 23:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants