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

bumping jsonschemafriend to 0.12.4 #37581

Merged

Conversation

stephane-airbyte
Copy link
Contributor

@stephane-airbyte stephane-airbyte commented Apr 25, 2024

jsonschema seems to have moved from a javascript redirect to a HTTP redirect for their DRAFT_4 URL. Unfortunately, jsonschemafriend doesn't handle HTTP redirect in 0.12.1, which causes our 2e2 source to dump inordinate amounts of logs. This fixes it.
The test GeneratorTest was failing prior to this change and is now passing

I believe this is the PR that fixes the issue (having an offline version of the supported stores, so there's no dependency on the internet at all)

Copy link

vercel bot commented Apr 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2024 5:19pm

Copy link
Contributor Author

stephane-airbyte commented Apr 25, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @stephane-airbyte and the rest of your teammates on Graphite Graphite

@stephane-airbyte stephane-airbyte marked this pull request as ready for review April 25, 2024 17:04
@stephane-airbyte stephane-airbyte requested a review from a team as a code owner April 25, 2024 17:04
@octavia-squidington-iv octavia-squidington-iv requested review from a team April 25, 2024 17:05
@stephane-airbyte stephane-airbyte changed the base branch from stephane/04-24-remove_unnecessary_calls_to_tolist_ to master April 25, 2024 17:14
@stephane-airbyte stephane-airbyte requested a review from a team as a code owner April 25, 2024 17:14
@stephane-airbyte stephane-airbyte force-pushed the stephane/04-25-bumping_jsonschemafriend_to_0.12.4 branch from 0c4118f to a366772 Compare April 25, 2024 17:15
@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Apr 25, 2024
@stephane-airbyte stephane-airbyte merged commit 401eee7 into master Apr 25, 2024
37 checks passed
@stephane-airbyte stephane-airbyte deleted the stephane/04-25-bumping_jsonschemafriend_to_0.12.4 branch April 25, 2024 17:36
FVidalCarneiro pushed a commit to AgiData/airbyte that referenced this pull request May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/e2e-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants