Skip to content

Add CLAUDE.md with build commands and architecture overview#136

Merged
saperi22 merged 2 commits into
mainfrom
add-claude-md
Apr 29, 2026
Merged

Add CLAUDE.md with build commands and architecture overview#136
saperi22 merged 2 commits into
mainfrom
add-claude-md

Conversation

@saperi22
Copy link
Copy Markdown
Collaborator

@saperi22 saperi22 commented Apr 28, 2026

Summary of changes

  • Add CLAUDE.md

AI Usage

Which AI Agent Was Used?

  • Copilot
  • Claude
  • Other (Type Name Here)

How was AI used?

Estimated AI Code Contribution

  • less than 30%
  • 30 - 60%
  • 60 - 100%

Checklist

  • Added a changelog entry
  • Tested and confirmed payment flows affected by this change are functioning as expected

Authors

List GitHub usernames for everyone who contributed to this pull request.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@saperi22 saperi22 requested a review from a team April 28, 2026 15:10
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

Dependency Review

✅ No vulnerabilities found.

Scanned Files

None

Copy link
Copy Markdown

@saralvasquez saralvasquez left a comment

Choose a reason for hiding this comment

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

Small question but otherwise looks good

Comment thread CLAUDE.md Outdated
### Versions

- compileSdk 36, minSdk 23, Java 11, Kotlin 1.9.20
- Current: `3.5.2-SNAPSHOT`; v2.x deprecated March 2026
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do we want the current version to be an unreleased snapshot or the current released version according to the CHANGELOG. Also will this get updated on release?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This doesn't get updated. I could just remove this part. I don't think this file needs to be the source on which version is the latest.
We could easily get it from the release tab/ tags.

Copy link
Copy Markdown

@saralvasquez saralvasquez left a comment

Choose a reason for hiding this comment

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

🎉

Copy link
Copy Markdown
Contributor

@jaxdesmarais jaxdesmarais left a comment

Choose a reason for hiding this comment

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

Thanks for adding this! The key classes was really helpful for me as someone that hasn't worked on browser switch much to get an overview of what it does. 😄

@saperi22 saperi22 merged commit eb0b9de into main Apr 29, 2026
10 checks passed
@saperi22 saperi22 deleted the add-claude-md branch April 29, 2026 21:21
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.

3 participants