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

Cleanup #11

Merged
merged 5 commits into from
Apr 16, 2024
Merged

Cleanup #11

merged 5 commits into from
Apr 16, 2024

Conversation

bhastings1019
Copy link
Collaborator

@bhastings1019 bhastings1019 commented Apr 9, 2024

📝 Description

Issue: PE-208

📋 List of Changes

  • update root readme for clarity
  • update example readmes for build, debug, deploy

✔️ Dev Complete Checklist

  • PR template filled out
  • Change is tested by submitter
  • PR follows all linting and coding standards
  • Github Issue exists (if applicable)
  • Team member has been assigned
  • At least one commit message is in Conventional Commit format

github-actions bot and others added 3 commits April 9, 2024 17:48
* update root readme for clarity
* update example readmes for build, debug, deploy
@bhastings1019 bhastings1019 requested a review from gherlein April 9, 2024 22:28

### Features

* Add tests for templates and nodejs examples ([#10](https://github.com/brightsign/dev-cookbook/issues/10)) ([3fe286c](https://github.com/brightsign/dev-cookbook/commit/3fe286c92a32e841b1900403d3f783565a1e9407))
Copy link
Contributor

Choose a reason for hiding this comment

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

This bullet starts with a capitalized letter. The others don't. Not huge, just nit picking.

README.md Outdated Show resolved Hide resolved
Co-authored-by: jdmedlin1 <[email protected]>
Signed-off-by: diegobrsign <[email protected]>
@bhastings1019 bhastings1019 merged commit 39db768 into main Apr 16, 2024
2 checks passed
@bhastings1019 bhastings1019 deleted the cleanup branch April 16, 2024 19:39
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.

4 participants