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 site list to sherlockproject.xyz/sites #2178

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Conversation

sdushantha
Copy link
Member

With these changes, the site list will be written to the sherlock-project/sherlockproject.xyz repo where the docs and everything is located. This way, we wont have to manually update the list on the website.

With these changes, the site list will be written to the
sherlock-project/sherlockproject.xyz repo where the docs and everything
is located. This way, we wont have to manually update the list on the
website.
@sdushantha
Copy link
Member Author

Note to self: blacklist site-list.py from the regression testing as its not needed. The reason why it got executed is because we set '**/*.py' as the rule

@sdushantha sdushantha merged commit 15180f9 into master Jun 20, 2024
15 checks passed
@ppfeister
Copy link
Member

When docs becomes a thing, would it be better moved over there? Or left here? Not sure myself

@ppfeister
Copy link
Member

Oh it's already a thing
And no subdomain now

Disregard

@ppfeister
Copy link
Member

Note to self: blacklist site-list.py from the regression testing as its not needed.

@sdushantha Ironic, because running tox locally now flags site-list.py as failed for the ruff lint

This raises the question of why isn't that tox item running on ci though....only local....

ppfeister added a commit that referenced this pull request Jun 24, 2024
Fixes hanging imports from #2151
Fixes bare f-string from #2178
@ppfeister ppfeister mentioned this pull request Jun 24, 2024
ppfeister added a commit that referenced this pull request Jun 24, 2024
Fixes hanging imports from #2151
Fixes bare f-string from #2178
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.

None yet

2 participants