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 develop with Issue and PR templates #186

Merged
merged 5 commits into from
Oct 12, 2023
Merged

Update develop with Issue and PR templates #186

merged 5 commits into from
Oct 12, 2023

Conversation

RuaridhMacd
Copy link
Collaborator

Description

Update develop with new README and templates

Fixes issue #

No associated issue

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Doesn't impact the code.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have run tests with my code to avoid compatibility issues
  • Any dependent changes have been merged and published in downstream modules

bennettm and others added 5 commits September 12, 2023 11:33
Squashed commit of the following:

commit a279e12
Author: bennettm <[email protected]>
Date:   Tue Sep 12 11:10:08 2023 -0400

    Update README.md

    Use a snapshot to help to understand

commit ba891db
Author: bennettm <[email protected]>
Date:   Tue Sep 12 10:59:19 2023 -0400

    Update README.md

    Correct a typo

commit f513e58
Author: bennettm <[email protected]>
Date:   Tue Sep 12 10:57:45 2023 -0400

    Update README.md

    Heading change

commit b4ec350
Author: bennettm <[email protected]>
Date:   Tue Sep 12 10:56:03 2023 -0400

    Update README.md

    Add link to a header

commit 84d208f
Author: bennettm <[email protected]>
Date:   Tue Sep 12 10:50:52 2023 -0400

    Update README.md

    Additions

commit 1bb6072
Author: Mary Bennett <[email protected]>
Date:   Tue Sep 12 10:32:20 2023 -0400

    Update README for a new way to run DOLPHYN
* Add issue and PR templates

* Update bug-report.yml with Manifest.toml file

* Correct validations tabbing

* Update feature request form

* Fix feature request formatting

* Fix YAML formatting error

* Fix checkbox labels

* Checkbox labels

* fix checkbox yaml error

---------

Co-authored-by: Mary Bennett <[email protected]>
Co-authored-by: Ruaridh Macdonald <[email protected]>
* Add updates to download DOLPHYN instance with submodule

* Add a colon

* Updating ReadMe

* Update README.md

---------

Co-authored-by: Mary Bennett <[email protected]>
Co-authored-by: Ruaridh Macdonald <[email protected]>
* Update README with team names

* Update issue and PR templates

- Fix assignment to DOLPHYN projects
- Explain need to change Manifest.toml extension
- Replace some duplicate descriptions with placeholder text so it remains after user starts typing

* Update pull_request_template.md

* Update formatting
@RuaridhMacd RuaridhMacd merged commit 4291080 into develop Oct 12, 2023
1 check failed
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