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

Fix schema.fieldsMatch #975

Closed
wants to merge 4 commits into from
Closed

Fix schema.fieldsMatch #975

wants to merge 4 commits into from

Conversation

roll
Copy link
Member

@roll roll commented Sep 17, 2024

Copy link

cloudflare-workers-and-pages bot commented Sep 17, 2024

Deploying datapackage with  Cloudflare Pages  Cloudflare Pages

Latest commit: 830fed6
Status: ✅  Deploy successful!
Preview URL: https://02e803fe.datapackage-6gp.pages.dev
Branch Preview URL: https://965-fix-fields-match.datapackage-6gp.pages.dev

View logs

@roll roll requested a review from peterdesmet September 17, 2024 13:49
@roll
Copy link
Member Author

roll commented Sep 17, 2024

Hi @peterdesmet, can you please take a look?

@roll roll changed the title Fix fields match Fix schema.fieldsMatch Sep 17, 2024
Copy link
Member

@peterdesmet peterdesmet left a comment

Choose a reason for hiding this comment

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

Actual change looks good (and is as discussed in 965).

Regarding the release:

  • I think we need to gather a number of fixes before we release a version. The release should be approved by the WG
  • Depending on how the website-build and other CLI is triggered, I would merge to a 2.1 branch, rather than to main, so we don't publish a 2.1 version that is not complete.

@@ -6,6 +6,14 @@ sidebar:

This document includes all meaningful changes made to the **Data Package standard**. It does not cover changes made to other documents like Recipes or Guides.

## v2.1

> Sep 17, 2024
Copy link
Member

Choose a reason for hiding this comment

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

To update: release should be approved by WG.

@@ -12,7 +12,7 @@ sidebar:
</tr>
<tr>
<th>Profile</th>
<td><a href="/profiles/2.0/datapackage.json">https://datapackage.org/profiles/2.0/datapackage.json</a></td>
<td><a href="/profiles/2.1/datapackage.json">https://datapackage.org/profiles/2.1/datapackage.json</a></td>
Copy link
Member

Choose a reason for hiding this comment

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

Might be good to make this table a Markdown table (assuming it is supported by the static website generator). That way the profile can just be referenced as:

<https://datapackage.org/profiles/2.1/datapackage.json>

Guaranteeing there is no discrepancy between the displayed URL and link

@roll
Copy link
Member Author

roll commented Sep 18, 2024

Thanks! I agree @peterdesmet

BTW @sapetti9 can you please ask OKFN to give me (gmail email) access to the datapackage pages project on Cloudflare so I can update the deployment approach (currently, we can only release from main).

@roll roll closed this Sep 24, 2024
@roll roll mentioned this pull request Sep 24, 2024
@roll roll deleted the 965/fix-fields-match branch September 25, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Difference beteween standard and JSON schema (fieldsMatch)
2 participants