Skip to content
This repository has been archived by the owner on Nov 8, 2024. It is now read-only.

chore: remove Circle-CI, fix GHA #2295

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

dblock
Copy link

@dblock dblock commented May 9, 2024

🚀 Why this change?

Build goes to green again.

  • Removes Circle-CI. Looks dead? Unused?
  • Moves smoke tests to GitHub Actions.
  • Run tests/lints/etc. on pull_request.
  • Renames jobs to tests (plural) to be consistent.
  • Locks sphinx doc dependencies to those compatible with v4.3.0.
  • https://github.com/ungrim97/Dredd-Hooks doesn't have functioning CI anymore (Travis CI is RIP), so links in docs removed
  • Fixes yarn docs:lint, but loses some anchors that no longer work and I couldn't find easy replacements for

See https://github.com/dblock/dredd/actions for everything passing.

📝 Related issues and Pull Requests

#2294

✅ What didn't I forget?

  • To write docs
  • To write tests
  • To put Conventional Changelog prefixes in front of all my commits and run npm run lint

node-version: [12.x]

steps:
- uses: supercharge/[email protected]
Copy link
Member

Choose a reason for hiding this comment

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

supercharge/[email protected] is not allowed to be used in apiaryio/dredd.

We're not going to be able to use this, perhaps it is worth removing this file/smoke tests from CI. Arguably merging this would still be better than current situation without smoke tests.

Copy link
Author

Choose a reason for hiding this comment

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

Not a problem. Updated to use GitHub's services, I was lazy and copy pasting from an older project ;)

Passing build in https://github.com/dblock/dredd/actions/runs/9043948597/job/24852077761.

@dblock dblock changed the title chore: remove Circle-CI chore: remove Circle-CI, fix GHA May 11, 2024
@dblock dblock requested a review from kylef May 11, 2024 13:23
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