Skip to content

Conversation

@erezrokah
Copy link
Member

Summary

Our docs show tables based on definition order which can be inconsistent and makes it harder to locate a table, see https://www.cloudquery.io/docs/plugins/sources/azure/tables:
image

This PR sorts the tables and relations before generating the docs


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

@erezrokah erezrokah changed the title feat(docs): Sort tables in docs feat(docs): Sort tables Jan 11, 2023
@github-actions github-actions bot added feat and removed feat labels Jan 11, 2023
@erezrokah erezrokah force-pushed the feat/sort_tables_in_docs branch from 1da842b to acc19c2 Compare January 11, 2023 11:14
@erezrokah erezrokah force-pushed the feat/sort_tables_in_docs branch from acc19c2 to c005d6e Compare January 11, 2023 11:16
@github-actions
Copy link

github-actions bot commented Jan 11, 2023

⏱️ Benchmark results

  • DefaultConcurrencyDFS-2 resources/s: 10,361
  • DefaultConcurrencyRoundRobin-2 resources/s: 10,990
  • Glob-2 ns/op: 145.6
  • TablesWithChildrenDFS-2 resources/s: 30,348
  • TablesWithChildrenRoundRobin-2 resources/s: 29,490
  • TablesWithRateLimitingDFS-2 resources/s: 28.25
  • TablesWithRateLimitingRoundRobin-2 resources/s: 861.4
  • BufferedScanner-2 ns/op: 10.04
  • LogReader-2 ns/op: 30.48

@kodiakhq kodiakhq bot merged commit 8a3bfad into cloudquery:main Jan 11, 2023
kodiakhq bot pushed a commit that referenced this pull request Jan 12, 2023
🤖 I have created a release *beep* *boop*
---


## [1.25.0](v1.24.2...v1.25.0) (2023-01-11)


### Features

* **docs:** Sort tables ([#599](#599)) ([8a3bfad](8a3bfad))
* **transformers:** Add support for `net.IP` ([#595](#595)) ([a420645](a420645))
* **transformers:** Add WithPrimaryKeys option ([#598](#598)) ([107006c](107006c))


### Bug Fixes

* Send resource validation errors to Sentry ([#601](#601)) ([5916516](5916516))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants