Skip to content
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

Update repo with cookiecutter #19

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Update repo with cookiecutter #19

merged 1 commit into from
Apr 27, 2023

Conversation

varsha888
Copy link
Contributor

Issue

Update repo with changes from cookiecutter

@varsha888 varsha888 self-assigned this Apr 26, 2023
@github-actions github-actions bot added size/XL Extra-Large PRs github_actions Pull requests that update github actions rust Pull requests that update rust code labels Apr 26, 2023
@github-actions
Copy link

github-actions bot commented Apr 26, 2023

❌ Unreviewed dependencies found

Crate Version Reviews (N/2) LoC Left-Pad Index Geiger Flags

@varsha888 varsha888 force-pushed the repo_update branch 2 times, most recently from 51c6910 to 855e3dd Compare April 27, 2023 01:22
@varsha888 varsha888 marked this pull request as ready for review April 27, 2023 01:25
Copy link
Collaborator

@nick-mobilecoin nick-mobilecoin 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 just minor nit on some missed rename

README.md Outdated Show resolved Hide resolved
@nick-mobilecoin nick-mobilecoin requested review from a team, awygle and samdealy and removed request for a team April 27, 2023 13:48
@varsha888 varsha888 merged commit 079653e into main Apr 27, 2023
18 checks passed
@varsha888 varsha888 deleted the repo_update branch April 27, 2023 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update github actions rust Pull requests that update rust code size/XL Extra-Large PRs
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants