Skip to content

[REFACTORING] ID tag expand abbreviation to full name#2991

Merged
battermann merged 4 commits intodevelopfrom
battermann/id-tag-refactoring
Jan 17, 2023
Merged

[REFACTORING] ID tag expand abbreviation to full name#2991
battermann merged 4 commits intodevelopfrom
battermann/id-tag-refactoring

Conversation

@battermann
Copy link
Contributor

@battermann battermann commented Jan 16, 2023

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@battermann battermann temporarily deployed to cachix January 16, 2023 12:53 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix January 16, 2023 12:53 — with GitHub Actions Inactive
@battermann battermann force-pushed the battermann/id-tag-refactoring branch from b0fbc7f to 9cacb3a Compare January 16, 2023 12:53
@battermann battermann temporarily deployed to cachix January 16, 2023 12:54 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix January 16, 2023 12:54 — with GitHub Actions Inactive
@battermann battermann marked this pull request as ready for review January 16, 2023 12:54
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 16, 2023
Copy link
Contributor

@elland elland left a comment

Choose a reason for hiding this comment

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

Some docs got mangled b/c of regex replaces.

Copy link
Member

@akshaymankar akshaymankar left a comment

Choose a reason for hiding this comment

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

Looks good apart from the comments being attacked by the regex

@battermann battermann temporarily deployed to cachix January 16, 2023 13:07 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix January 16, 2023 13:07 — with GitHub Actions Inactive
@battermann battermann force-pushed the battermann/id-tag-refactoring branch from 10f054d to 53815d5 Compare January 16, 2023 13:07
@battermann battermann temporarily deployed to cachix January 16, 2023 13:07 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix January 16, 2023 13:07 — with GitHub Actions Inactive
@elland
Copy link
Contributor

elland commented Jan 16, 2023

Love seeing this kind of PR <3

@battermann battermann temporarily deployed to cachix January 16, 2023 16:16 — with GitHub Actions Inactive
@battermann battermann temporarily deployed to cachix January 16, 2023 16:16 — with GitHub Actions Inactive
@battermann battermann merged commit b2c183a into develop Jan 17, 2023
@battermann battermann deleted the battermann/id-tag-refactoring branch January 17, 2023 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments