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.

Sign up for GitHub

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

Jump to bottom

updates application structure #10

Merged
merged 14 commits into from
Jun 10, 2024
Merged

updates application structure #10

merged 14 commits into from
Jun 10, 2024

Conversation

chrabyrd
Copy link
Contributor

No description provided.

@chrabyrd chrabyrd requested a review from robgaston April 11, 2024 05:26
@chrabyrd chrabyrd changed the title updates application struture updates application structure Apr 16, 2024
@robgaston robgaston mentioned this pull request May 21, 2024
Merged
robgaston
robgaston requested changes May 23, 2024
View reviewed changes
Copy link
Member

@robgaston robgaston left a comment

Choose a reason for hiding this comment

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

@chrabyrd could you please rebase this branch to include the latest changes from main?

@chrabyrd chrabyrd requested a review from robgaston May 29, 2024 07:15
robgaston
robgaston requested changes Jun 4, 2024
View reviewed changes
Copy link
Member

@robgaston robgaston left a comment

Choose a reason for hiding this comment

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

@chrabyrd this is looking very close, but we should look at updating the README.md and moving from yarn to npm. thanks!

README.txt Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
@chrabyrd
Copy link
Contributor Author

chrabyrd commented Jun 4, 2024

@chrabyrd this is looking very close, but we should look at updating the README.md and moving from yarn to npm. thanks!

Ah yes -- this was done before the cutover from yarn to npm. Will update! 👍

@robgaston robgaston merged commit 599688e into main Jun 10, 2024
5 of 6 checks passed
@robgaston robgaston deleted the cbyrd-update-application-structure branch June 10, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robgaston robgaston robgaston approved these changes

Assignees

@robgaston robgaston

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chrabyrd @robgaston