Skip to content

chore(ci): Lint markdown files#119

Merged
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
erezrokah:chore/add_markdown_lint
Sep 18, 2022
Merged

chore(ci): Lint markdown files#119
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
erezrokah:chore/add_markdown_lint

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

Adds linting to markdown files and fixes our README and Contributing guide (which some redundant/stale information, dead links, etc)


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 ✅

@@ -0,0 +1,109 @@
github
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is the same as https://github.com/cloudquery/cloudquery/tree/bac87adb620104eb4eb7fcc715cfe6c44c96f8f5/.github/styles.

As a follow up we could create a dedicated repository just for those files and publish the styles as a zip artifact, then use a URL instead of a directory


The following is a set of guidelines for contributing to this repository.


Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

GitHub automatically generates table of content for markdown files
image


CloudQuery has a pluggable architecture and is using the go-plugin to load, run, and communicate between plugins via gRPC. T

![architecture overview](https://docs.cloudquery.io/assets/images/cloudquery-architecture-02b1d1162883cd02510db2cb75c29d27.png)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This image link is broken

* Downloading, verifying, and running plugins
* Running policy packs

#### CloudQuery Plugins and CLI [Mono Repo](https://github.com/cloudquery/cloudquery):
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Duplicate reference to the mono repo


CloudQuery has a pluggable architecture and is using grpc (go-grpc) to communicate between source plugins, cli and destination plugins. To develop a new plugin for CloudQuery, you don’t need to understand the inner workings of grpc as those are abstracted away via the [plugin-sdk](#cloudquery-plugin-sdk-repo).

CloudQuery has a pluggable architecture and is using the go-plugin to load, run, and communicate between plugins via gRPC. T
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

go-plugin is 💀


## How Can I Contribute?

### Reporting Bugs
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This seems like a duplication of information that should be in our issue templates


### Your First Code Contribution

Unsure where to begin contributing to CloudQuery? You can start by looking through these `beginner` and `help-wanted` issues:
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We haven't used beginner anywhere. Also I think searching for 1 label should be enough. good first issue is a default label all GitHub repositories have

#### Local Development

CloudQuery has the ability to be run locally with a corresponding local postgres database. To get it up and running follow the following instructions:
* [Building and Running the plugin Locally](../docs/index.md)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Broken link

README.md Outdated
@@ -1,20 +1,18 @@
CloudQuery Plugin SDK ![License](https://img.shields.io/github/license/cloudquery/cloudquery?style=flat-square)
=======================
# CloudQuery Plugin SDK ![License](https://img.shields.io/github/license/cloudquery/cloudquery?style=flat-square)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Use the same header format (# instead of =======================)


## Supported plugins

https://www.cloudquery.io/plugins
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

GitHub knows how to render those as links, but better to be explicit

@erezrokah
Copy link
Copy Markdown
Member Author

Ah noticed some of the README changes are included in https://github.com/cloudquery/plugin-sdk/pull/118/files, so this PR should be merged after

@erezrokah erezrokah force-pushed the chore/add_markdown_lint branch from d8d7d6c to 3b48a2f Compare September 18, 2022 15:12
@kodiakhq kodiakhq bot merged commit b2287f8 into cloudquery:main Sep 18, 2022
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.

2 participants