Skip to content

chore: Add examples to workspace#283

Merged
sugyan merged 2 commits intoatrium-rs:mainfrom
DrChat:examples
Feb 15, 2025
Merged

chore: Add examples to workspace#283
sugyan merged 2 commits intoatrium-rs:mainfrom
DrChat:examples

Conversation

@DrChat
Copy link
Contributor

@DrChat DrChat commented Feb 14, 2025

Based on this discussion.

It's hard to keep examples up-to-date with the current state of the repository because they are explicitly targeting the last released version.
However, it's fairly trivial for users to discover the examples for a particular release by looking at the git tags.

Given that, it seems reasonable to just add the examples to the workspace and direct users to use git tags to discover the examples for the previous release.

@sugyan sugyan self-requested a review February 15, 2025 14:19
Copy link
Member

@sugyan sugyan left a comment

Choose a reason for hiding this comment

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

Thank you very much! I agree with this.
It seems that CI is making an error, probably because something I wrote a long time ago was an old formatter. 🙇
I think running cargo fmt-no-gen will fix it.

@DrChat DrChat requested a review from sugyan February 15, 2025 17:03
@DrChat
Copy link
Contributor Author

DrChat commented Feb 15, 2025

@sugyan Done, thanks!

Copy link
Member

@sugyan sugyan left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@sugyan sugyan merged commit ceac515 into atrium-rs:main Feb 15, 2025
13 checks passed
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.

2 participants