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 Commands #633

Merged
merged 2 commits into from
Dec 20, 2022
Merged

Update Commands #633

merged 2 commits into from
Dec 20, 2022

Conversation

Erin-Cecele
Copy link
Collaborator

make other small text changes to match docs README

Thank you for contributing to Quire! Please complete the form below to submit your pull request for review.

For the Title of this pull request, please use the format "Type/Issue-#: Brief description." For Type, the options are Fix, Feature, Docs, or Chore. Issue-# is only needed if this pull request addresses an exisiting issue.

Checklist

Please put an X within the brackets that apply [X].

  • I have read the CONTRIBUTING.md file

  • I have made my changes in a new branch and not directly in the main branch

  • This pull request is ready for final review by the Quire team

Is this pull request related to an open issue? If so, what is the issue number?

No

Please briefly describe the goal of this pull request and how it may impact Quire's functionality.

Update commands list in README

Please describe the changes you made, and call out any details you think are particularly relevant for the Quire team to note in their review.

In addition to commands, also made changes to some of the text so it matches quire-docs README

Does this pull request necessitate changes to Quire's documentation?

No

Include screenshots of before/after if applicable.

Additional Comments

@geealbers, while you review my PR in the quire-docs repo related to the CLI, I want to make sure this README accurately reflects the list of available commands

make other small text changes to match docs README
Copy link
Member

@geealbers geealbers left a comment

Choose a reason for hiding this comment

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

One small change

README.md Outdated Show resolved Hide resolved
@Erin-Cecele
Copy link
Collaborator Author

@geealbers I made the correction

@Erin-Cecele Erin-Cecele merged commit c53c8c9 into main Dec 20, 2022
@Erin-Cecele Erin-Cecele deleted the Erin-Cecele-patch-2 branch December 20, 2022 17:16
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