Skip to content

Code ADR for mithril-common #2343

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

Merged
merged 3 commits into from
Mar 6, 2025
Merged

Code ADR for mithril-common #2343

merged 3 commits into from
Mar 6, 2025

Conversation

Alenar
Copy link
Collaborator

@Alenar Alenar commented Feb 26, 2025

Content

This PR add a "code ADR" file to the mithril-common project in order to details project specific decisions that would be to specific to fit our website ADR blog.

Pre-submit checklist

  • Branch
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Issue(s)

Relates to #2342

@Alenar Alenar self-assigned this Feb 26, 2025
Copy link

github-actions bot commented Feb 26, 2025

Test Results

    3 files  ±0     55 suites  ±0   10m 9s ⏱️ -14s
1 683 tests ±0  1 683 ✅ ±0  0 💤 ±0  0 ❌ ±0 
2 075 runs  ±0  2 075 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit f63bce2. ± Comparison against base commit 380cbba.

♻️ This comment has been updated with latest results.

@Alenar Alenar temporarily deployed to testing-preview February 26, 2025 15:41 — with GitHub Actions Inactive
@Alenar Alenar force-pushed the djo/2342/code-adr branch from 5c72962 to eb6ff85 Compare March 5, 2025 10:54
@Alenar Alenar temporarily deployed to testing-preview March 5, 2025 11:03 — with GitHub Actions Inactive
@Alenar Alenar force-pushed the djo/2342/code-adr branch from eb6ff85 to b7b5691 Compare March 5, 2025 11:06
@Alenar Alenar temporarily deployed to testing-preview March 5, 2025 11:14 — with GitHub Actions Inactive
@Alenar Alenar force-pushed the djo/2342/code-adr branch from b7b5691 to fd0a853 Compare March 5, 2025 13:57
@Alenar Alenar temporarily deployed to testing-preview March 5, 2025 14:05 — with GitHub Actions Inactive
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Alenar added 3 commits March 6, 2025 10:06
- We want to have only one file to avoid fragmentations (some ADR can
  concerns multiple projects).
- The name is to help differentiate this file from the website blog ADR
  (that record more structural decisions).
- specify that all check jobs must have passed instead of only clippy
- specify that a ADR blog post or a DevADR entry should be added if
  relevant
- specify that no new TODOs should be added to keep the project clean
@Alenar Alenar force-pushed the djo/2342/code-adr branch from fd0a853 to f63bce2 Compare March 6, 2025 09:06
@Alenar Alenar temporarily deployed to testing-preview March 6, 2025 09:15 — with GitHub Actions Inactive
@Alenar Alenar merged commit 310dd2f into main Mar 6, 2025
38 checks passed
@Alenar Alenar deleted the djo/2342/code-adr branch March 6, 2025 10:51
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.

4 participants