Skip to content

[Fix] Refactor nightly skip logic#8

Merged
arjkesh merged 9 commits intoNVIDIA:mainfrom
arjkesh:readme_update
Dec 18, 2025
Merged

[Fix] Refactor nightly skip logic#8
arjkesh merged 9 commits intoNVIDIA:mainfrom
arjkesh:readme_update

Conversation

@arjkesh
Copy link
Copy Markdown
Collaborator

@arjkesh arjkesh commented Dec 15, 2025 •
edited
Loading

Description

  • Fix nightly skip logic to ensure we don't rerun builds on the same source commit by default

CI Configuration

config:
  build: false
  # valid options are "ops" and "benchmark"
  test: []

Checklist

  • Documentation updated (if needed)
  • CI configuration reviewed

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Dec 15, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@arjkesh
Copy link
Copy Markdown
Collaborator Author

arjkesh commented Dec 15, 2025

/ok to test 503d7b0

@arjkesh
Copy link
Copy Markdown
Collaborator Author

arjkesh commented Dec 15, 2025

/ok to test ce49a9b

@arjkesh arjkesh enabled auto-merge (squash) December 15, 2025 17:23
@arjkesh
Copy link
Copy Markdown
Collaborator Author

arjkesh commented Dec 15, 2025

/ok to test 3e7a696

xjmxyt
xjmxyt reviewed Dec 16, 2025
View reviewed changes
Comment thread README.md Outdated
Comment thread .github/workflows/tilegym-ci.yml
@arjkesh arjkesh changed the title Update instructions for running with pre-built docker image and fix nightly skip logic [Fix] Refactor nightly skip logic Dec 17, 2025
@arjkesh
Copy link
Copy Markdown
Collaborator Author

arjkesh commented Dec 17, 2025

/ok to test 8fa659b

@arjkesh arjkesh disabled auto-merge December 17, 2025 21:28
xjmxyt
xjmxyt approved these changes Dec 18, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@xjmxyt xjmxyt left a comment

Choose a reason for hiding this comment

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

LGTM, thx.

@arjkesh
Copy link
Copy Markdown
Collaborator Author

arjkesh commented Dec 18, 2025

/ok to test 088f577

@arjkesh arjkesh merged commit 52cb5bf into NVIDIA:main Dec 18, 2025
8 checks passed
@arjkesh arjkesh deleted the readme_update branch December 18, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xjmxyt xjmxyt xjmxyt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arjkesh @xjmxyt