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

chore: Schema test for graphql #648

Merged
merged 6 commits into from
Apr 19, 2024
Merged

Conversation

oskogstad
Copy link
Collaborator

@oskogstad oskogstad commented Apr 17, 2024

Automatic testing for changes in GraphQL schema (same as OpenAPI)
Renamed docs/swagger to docs/schema to host the two files in the same npm package.
Renamed npm package to dialogporten-schema

Automatic versioning/publishing of this npm package is being worked on in another branch

Related Issue(s)

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

  • Documentation is updated (either in docs-directory, Altinnpedia or a separate linked PR in altinn-studio-docs., if applicable)

@oskogstad oskogstad requested a review from a team as a code owner April 17, 2024 12:44
arealmaas
arealmaas previously approved these changes Apr 17, 2024
@oskogstad oskogstad force-pushed the chore/schema-test-for-graphql branch from 899d24d to 08e9cbb Compare April 17, 2024 14:19
@oskogstad oskogstad force-pushed the chore/schema-test-for-graphql branch from 457f8b8 to 463d134 Compare April 18, 2024 08:58
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@oskogstad oskogstad merged commit 343b0e9 into main Apr 19, 2024
14 checks passed
@oskogstad oskogstad deleted the chore/schema-test-for-graphql branch April 19, 2024 13:04
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.

2 participants