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

clean up #1295

Merged
merged 26 commits into from
Sep 26, 2024
Merged

clean up #1295

merged 26 commits into from
Sep 26, 2024

Conversation

gfursin
Copy link
Contributor

@gfursin gfursin commented Sep 26, 2024

No description provided.

arjunsuresh and others added 25 commits September 17, 2024 13:49
   - added cmind.core.debug to make it easier to debug CM automations
   - added env CM_CORE_SKIP_FIX_REPO_PATH to skip fixing non-existent repo paths
## V2.3.6.1
   - added cmind.core.debug to make it easier to debug CM automations
- added env CM_CORE_SKIP_FIX_REPO_PATH to skip fixing non-existent repo
paths
added core.debug and env CM_CORE_SKIP_FIX_REPO_PATH (#1293)
@gfursin gfursin requested a review from a team as a code owner September 26, 2024 13:29
Copy link

github-actions bot commented Sep 26, 2024

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

- added cmind.core.debug to make it easier to debug CM automations
- added env CM_CORE_SKIP_FIX_REPO_PATH to skip fixing non-existent repo
paths
@gfursin
Copy link
Contributor Author

gfursin commented Sep 26, 2024

Hi @arjunsuresh . Did I understand you correctly, that it's normal that the README check fails, here? If yes, should we merge this PR? I also just realized that I didn't create PR from my fork to dev but to main directly ...

@arjunsuresh
Copy link
Contributor

yes Grigori. I think it'll be good to just clean up all the broken links (there are many false positives like the linkedin URLs) and then restrict the check to only the links coming from the PR. Currently it is checking the entire repository.

@gfursin
Copy link
Contributor Author

gfursin commented Sep 26, 2024

I see. Sure, that makes sense! I will ignore this test for now but let's wait for all other tests before merging other PRs ;) ...
Thanks!!!

@gfursin gfursin merged commit b3bc147 into dev Sep 26, 2024
29 of 30 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants