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

API: Cities #96

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

API: Cities #96

wants to merge 15 commits into from

Conversation

sgerrand
Copy link
Contributor

@sgerrand sgerrand commented Jul 12, 2024

💁 These changes:

  • add support for the Cities API
  • alters DuffelAPI::Resources::Airport.new.city to return a DuffelAPI::Resource::City

⚠️ The test suite wouldn't run so to get that passing I've also made the following changes:

  • updated gc_ruboconfig, rubocop gems
  • fixed various offenses raised by RuboCop
  • fixed a silently broken test case
  • bumped the minimum Ruby version to 2.7 (this version is EOL)

@sgerrand sgerrand requested a review from a team as a code owner July 12, 2024 16:01
@sgerrand
Copy link
Contributor Author

👋 @ulissesalmeida, are you able to review these changes?

Time moves on and the ages in these examples are no longer valid. These
changes replace static ages with calculated equivalents so that they can
be rerun at any time with confidence.
Copy link
Contributor

@ulissesalmeida ulissesalmeida left a comment

Choose a reason for hiding this comment

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

@sgerrand Long time my friend!

That is tons of work!
Thank you.

Looks good to me!

@sgerrand
Copy link
Contributor Author

Too long! Hope that you're doing well. 💝

sgerrand and others added 14 commits September 13, 2024 15:17
Dependabot: Manage GitHub Actions updates
Gemspec/DevelopmentDependencies: Specify development dependencies in
Gemfile.
Pull test case out of setup.
- Remove Ruby 2.6: rubocop >= 1.51.0 depends on Ruby >= 2.7.0
- Add modern, generally available Ruby versions: 3.2, 3.3
Style/SuperArguments: Call super without arguments and parentheses when
the signature is identical.
Style/RedundantInterpolation: Prefer to_s over string interpolation.
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 2 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v2...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Examples: time aware passenger ages
- Style/SuperArguments: Call super without arguments and parentheses
  when the signature is identical.
- Layout/EmptyLinesAroundBlockBody: Extra empty line detected at block
  body end. (https://rubystyle.guide#empty-lines-around-bodies)
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