Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

update contributing.md #449

Merged
merged 4 commits into from
Apr 4, 2022
Merged

update contributing.md #449

merged 4 commits into from
Apr 4, 2022

Conversation

kshitij12345
Copy link
Contributor

@kshitij12345 kshitij12345 commented Mar 24, 2022 •
edited
Loading

Fixes: #447

@kshitij12345
Copy link
Contributor Author

cc: @allisonvacanti

@alliepiper alliepiper mentioned this pull request Mar 25, 2022
Closed
alliepiper
alliepiper suggested changes Mar 25, 2022
View reviewed changes
Copy link
Collaborator

@alliepiper alliepiper left a comment

Choose a reason for hiding this comment

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

See comments in #447 (comment)

@kshitij12345
Copy link
Contributor Author

Done. Thanks!

alliepiper
alliepiper approved these changes Apr 4, 2022
View reviewed changes
Copy link
Collaborator

@alliepiper alliepiper left a comment

Choose a reason for hiding this comment

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

LGTM -- thanks!

@alliepiper alliepiper merged commit 998ff61 into NVIDIA:main Apr 4, 2022
@alliepiper alliepiper added this to the 1.17.0 milestone Apr 4, 2022
@alliepiper alliepiper added the only: docs Documentation changes only. Doesn't need code CI. label Apr 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alliepiper alliepiper alliepiper approved these changes

Assignees
No one assigned
Labels
only: docs Documentation changes only. Doesn't need code CI.
Projects
None yet
Milestone
1.17.0
Development

Successfully merging this pull request may close these issues.

Link pointing to Thrust Development Model in CONTRIBUTING.md is invalid (gives 404)
2 participants
@kshitij12345 @alliepiper