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

Accident, please ignore. #405

Closed
rwgk opened this issue Jan 16, 2025 · 3 comments
Closed

Accident, please ignore. #405

rwgk opened this issue Jan 16, 2025 · 3 comments
Labels
triage Needs the team's attention

Comments

@rwgk
Copy link
Collaborator

rwgk commented Jan 16, 2025

In PR #3201 I tried to add taplo-lint to pre-commit. It works interactively, but has been failing for ~2 days in the CCCL CI (error below). I'll comment it out for the moment to unblock #3201. This issue is to remind us to revisit. Hopefully the "Temporary failure in name resolution" will fix itself and we can just uncomment taplo-lint in .pre-commit-config.yaml.

fix end of files.........................................................Passed
mixed line ending........................................................Passed
trim trailing whitespace.................................................Passed
clang-format.............................................................Passed
ruff.....................................................................Passed
ruff-format..............................................................Passed
taplo-lint...............................................................Failed
- hook id: taplo-lint
- exit code: 1

 WARN taplo: failed to fetch catalog error=error sending request for url (https://www.schemastore.org/api/json/catalog.json): error trying to connect: dns error: failed to lookup address information: Temporary failure in name resolution

Caused by:
    0: error trying to connect: dns error: failed to lookup address information: Temporary failure in name resolution
    1: dns error: failed to lookup address information: Temporary failure in name resolution
    2: failed to lookup address information: Temporary failure in name resolution

taplo-format.............................................................Passed
codespell................................................................Passed
mypy.....................................................................Passed
@github-actions github-actions bot added the triage Needs the team's attention label Jan 16, 2025
@rwgk
Copy link
Collaborator Author

rwgk commented Jan 16, 2025

Oh no, sorry, I got the wrong repo!

@rwgk rwgk closed this as completed Jan 16, 2025
@rwgk rwgk changed the title [FEA]: Enable taplo-lint pre-commit Accident, please ignore. Jan 16, 2025
@leofang
Copy link
Member

leofang commented Jan 16, 2025

hahaha no worries, would you like me to transfer the issue to cccl repo?

@leofang
Copy link
Member

leofang commented Jan 16, 2025

oh I see you already opened one NVIDIA/cccl#3426

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Needs the team's attention
Projects
None yet
Development

No branches or pull requests

2 participants