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

feat(dynamite): allow per spec configuration #1174

Merged
merged 3 commits into from
Dec 22, 2023

Conversation

Leptopoda
Copy link
Member

I don't think we should merge this without using it.
I just wanted to get feedback and avoid letting the branch just rot on my machine

@Leptopoda Leptopoda force-pushed the feat/dynamite/per_spec_config branch from 5d1346d to a758d6a Compare November 22, 2023 17:00
@provokateurin
Copy link
Member

I think for testing this we could change some of the rules discussed in #1245 for a single (separate) spec.

@Leptopoda
Copy link
Member Author

My long term goal was to also generate doc comments for the generated Client so we can just enable public_member_api_docs while testing our documentation at the same time.

@provokateurin
Copy link
Member

Another real world use case for this would be annotating the news, notes and uppush code as experimental because those are specs are written by hand and are not complete.

@Leptopoda
Copy link
Member Author

I'll rebase and take a look.

@Leptopoda Leptopoda force-pushed the feat/dynamite/per_spec_config branch 3 times, most recently from 2f393a7 to 6924b43 Compare December 22, 2023 11:37
@Leptopoda
Copy link
Member Author

I've added the experimental annotation.
It's currently based on #1337 so tests might fail due to #1343

@Leptopoda Leptopoda linked an issue Dec 22, 2023 that may be closed by this pull request
@Leptopoda Leptopoda force-pushed the feat/dynamite/per_spec_config branch from 6924b43 to d031856 Compare December 22, 2023 14:46
@Leptopoda
Copy link
Member Author

should be ready for review @provokateurin

packages/dynamite/dynamite/README.md Show resolved Hide resolved
packages/nextcloud/build.yaml Outdated Show resolved Hide resolved
@Leptopoda Leptopoda force-pushed the feat/dynamite/per_spec_config branch from d031856 to 5a5dd65 Compare December 22, 2023 15:46
@Leptopoda Leptopoda merged commit 47d9e33 into main Dec 22, 2023
8 checks passed
@Leptopoda Leptopoda deleted the feat/dynamite/per_spec_config branch December 22, 2023 17:36
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.

Mark generated news, notes and uppush code as experimental
2 participants