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: refactor tags #317

Merged
merged 1 commit into from
Apr 26, 2022
Merged

fix: refactor tags #317

merged 1 commit into from
Apr 26, 2022

Conversation

jstirnaman
Copy link
Contributor

@jstirnaman jstirnaman commented Apr 22, 2022

  • Moves tags and x-tagGroups to separate files in src/oss and src/cloud.
  • Move x-traitTags to tags.yml.
  • Remove unused extract-tags
  • Tags will be collected in the Docs pipeline.

This PR moves openapi tag descriptions, x-traitTag , x-tagGroups to new files - mostly just a small refactor for readability as the list grows. I'm not sure if anyone except Docs uses them. We temporarily managed them in the Docs repo, but this PR along with influxdata/docs-v2#3977 assume openapi will be the primary source. In the Docs pipeline, we use OpenAPI CLI to extract all tags and populate the "All Endpoints" group.

@jstirnaman jstirnaman marked this pull request as draft April 22, 2022 21:54
@jstirnaman jstirnaman force-pushed the fix/openapi-tags branch 9 times, most recently from 226ba4b to 30ba5de Compare April 22, 2022 22:50
@jstirnaman jstirnaman marked this pull request as ready for review April 22, 2022 22:52
@jstirnaman jstirnaman force-pushed the fix/openapi-tags branch 4 times, most recently from 8aa8595 to 6bb7a49 Compare April 25, 2022 16:42
- restore and reorg tags and tag groups.
- cleanup paths
- rename invocable to invokable to match preferred spelling in other docs.
@jstirnaman jstirnaman merged commit ea52b27 into master Apr 26, 2022
@jstirnaman jstirnaman deleted the fix/openapi-tags branch April 26, 2022 18:55
bednar added a commit to influxdata/influxdb-client-java that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-csharp that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-php that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-python that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-ruby that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-dart that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-swift that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-js that referenced this pull request Apr 27, 2022
bednar added a commit to influxdata/influxdb-client-js that referenced this pull request Apr 28, 2022
bednar added a commit to influxdata/influxdb-client-swift that referenced this pull request Apr 28, 2022
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