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.

Sign up for GitHub

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

Jump to bottom

Delete check-links.yml #50

Merged
merged 2 commits into from
Jul 29, 2022
Merged

Delete check-links.yml #50

merged 2 commits into from
Jul 29, 2022

Conversation

dalonsoa
Copy link
Collaborator

Description

Checks are failing for issues outside our control. To re-enable in the future. See tcort/markdown-link-check#201

Fixes # None

Type of change

  • New feature (non-breaking change which adds functionality)
  • Optimization (back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)

Key checklist:

N/A

  • Make sure you've run the pre-commit checks: $ pre-commit run -a
  • All tests pass: $ poetry run pytest

Further checks:

N/A

  • Code is commented, particularly in hard-to-understand areas
  • Tests added that prove fix is effective or that feature works

@dalonsoa dalonsoa requested a review from davidorme July 29, 2022 13:50
@davidorme davidorme closed this Jul 29, 2022
@davidorme davidorme reopened this Jul 29, 2022
@robewers01 robewers01 self-requested a review July 29, 2022 13:50
@dalonsoa dalonsoa merged commit c438907 into develop Jul 29, 2022
@dalonsoa dalonsoa deleted the dalonsoa-patch-1 branch July 29, 2022 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidorme davidorme davidorme approved these changes

@robewers01 robewers01 Awaiting requested review from robewers01

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dalonsoa @davidorme @alexdewar