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

Fix auto merge conflict 9372 #9373

Merged
merged 2 commits into from
Oct 4, 2023

Conversation

mattahrens
Copy link
Collaborator

Do not squash commits

…nvidia… (NVIDIA#9362)

* Removing User Guide pages that will be source of truth on docs.nvidia.com

Signed-off-by: mattahrens <matthewahrens@gmail.com>

* Updating links in README to point to docs.nvidia.com User Guide

Signed-off-by: mattahrens <matthewahrens@gmail.com>

* Fixing broken links in other pages for User Guide updates

Signed-off-by: mattahrens <matthewahrens@gmail.com>

* Fixing broken links in supported ops page

Signed-off-by: mattahrens <matthewahrens@gmail.com>

* Updating TypeChecks to keep in line with supported_ops.md

Signed-off-by: mattahrens <matthewahrens@gmail.com>

---------

Signed-off-by: mattahrens <matthewahrens@gmail.com>
Signed-off-by: mattahrens <matthewahrens@gmail.com>
@mattahrens mattahrens requested a review from jlowe October 3, 2023 20:44
@mattahrens
Copy link
Collaborator Author

build

Copy link
Collaborator

@razajafri razajafri left a comment

Choose a reason for hiding this comment

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

Why didn't we add [skip ci]? If it's because of the two scala files won't that be caught early on in our checks?

@jlowe jlowe merged commit 08f083b into NVIDIA:branch-23.12 Oct 4, 2023
30 checks passed
@jlowe
Copy link
Member

jlowe commented Oct 4, 2023

Why didn't we add [skip ci]?

Yes, it's because of the code changes. Yes, arguably one could say the git workflow checks that just compile but don't run any tests are sufficient for this change, but that becomes a slippery slope if people start deciding to skip CI even with code changes because they're convinced it won't matter. It wasn't critical to get this in ASAP, so IMO running through CI was fine.

@sameerz sameerz added the task Work required that improves the product but is not user facing label Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Work required that improves the product but is not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants