Skip to content

[WPB-7222] (part 2) add coding-conventions.md to developer docs#4006

Merged
mdimjasevic merged 3 commits intodevelopfrom
mangoiv/wpb-7222/document-cs-change
Apr 18, 2024
Merged

[WPB-7222] (part 2) add coding-conventions.md to developer docs#4006
mdimjasevic merged 3 commits intodevelopfrom
mangoiv/wpb-7222/document-cs-change

Conversation

@MangoIV
Copy link
Contributor

@MangoIV MangoIV commented Apr 17, 2024

https://wearezeta.atlassian.net/browse/WPB-7222

Checklist

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

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Apr 17, 2024
@MangoIV MangoIV requested review from fisx and mdimjasevic April 17, 2024 08:42
Copy link
Contributor

@fisx fisx left a comment

Choose a reason for hiding this comment

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

i think we decided to not have one of these because nobody is going to keep it in memory anyway, but i'm still all in favor. :)

@mdimjasevic mdimjasevic merged commit 011fd4b into develop Apr 18, 2024
@mdimjasevic mdimjasevic deleted the mangoiv/wpb-7222/document-cs-change branch April 18, 2024 07:43
@echoes-hq echoes-hq bot added the echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. label Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. 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