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.

Sign up for GitHub

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

Jump to bottom

chore: Remove user #41

Closed
wants to merge 1 commit into from
Closed

chore: Remove user #41

wants to merge 1 commit into from

Conversation

nhhagen
Copy link
Member

@nhhagen nhhagen commented Apr 15, 2024

No description provided.

@nhhagen nhhagen requested a review from a team as a code owner April 15, 2024 12:12
@nhhagen nhhagen enabled auto-merge (squash) April 15, 2024 12:12
@bendiknesbo
Copy link
Contributor

Why do we want to remove people from this list?
If the user/employee is mentioned in a document (such as a post-mortem), then this Vale-update would start to break existing post-mortems (except for the fact that we mostly do validate only on changed file).

@bendiknesbo bendiknesbo mentioned this pull request Apr 15, 2024
Merged
@nhhagen
Copy link
Member Author

nhhagen commented Apr 15, 2024

Closing in favour of #43

@nhhagen nhhagen closed this Apr 15, 2024
auto-merge was automatically disabled April 15, 2024 13:17

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pravindahal pravindahal pravindahal approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nhhagen @bendiknesbo @pravindahal