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.

Sign up for GitHub

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

Jump to bottom

ci: add lint, links, gosec and depbot #7

Merged
merged 7 commits into from
Mar 29, 2022
Merged

ci: add lint, links, gosec and depbot #7

merged 7 commits into from
Mar 29, 2022

Conversation

pratikbin
Copy link
Member

@pratikbin pratikbin commented Mar 25, 2022 •
edited
Loading

  • Lint CI for linting project
  • Gosec golang security checker
  • Links ci for checking broker links in docs
  • Dependabot for dep update
  • Codeql for semantic code analysis engine

This was referenced Mar 25, 2022
Closed
Closed
This was linked to issues Mar 28, 2022
Add static security scanner and linter #5
Closed
Add Semantic Pull Request #3
Closed
@Naman2706
Copy link
Contributor

@pratikbin please rebase

@pratikbin
ci: add lint, links, gosec and depbot
2596899 
- Lint ci for linting project
- Gosec golang security checker
- Links ci for checking broker links in docs
- Dependabot for dep updation
@pratikbin
ci: add auto rebase ci
3264d86
@pratikbin
ci: label based rebase ci trigger
c7a7f7d
@github-actions github-actions bot removed the rebase label Mar 28, 2022
@pratikbin
Copy link
Member Author

rebase ci will force push, @aman00323 How dangerous it is?

@aman00323
Copy link
Contributor

@pratikbin can't rely on right now

@pratikbin
Copy link
Member Author

pratikbin commented Mar 28, 2022 •
edited
Loading

You mean remove it ?

@aman00323
Copy link
Contributor

Yes, force push might ruin someone's work.

@pratikbin
Copy link
Member Author

pratikbin commented Mar 28, 2022 •
edited
Loading

Thought so. Checked for flags for not forcing push, but didn't find any. Will remove it

@pratikbin
Copy link
Member Author

Squash merge it please

@aman00323 aman00323 merged commit 3ee47c8 into main Mar 29, 2022
@aman00323 aman00323 deleted the feat/basic-ci branch March 29, 2022 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shyamvyas5 shyamvyas5 shyamvyas5 approved these changes

@Naman2706 Naman2706 Naman2706 approved these changes

@aman00323 aman00323 Awaiting requested review from aman00323

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add static security scanner and linter Add Semantic Pull Request
4 participants
@pratikbin @Naman2706 @aman00323 @shyamvyas5