Skip to content

Feat/GitHub provider#2843

Closed
carvido1 wants to merge 13 commits intoexternal-secrets:mainfrom
carvido1:feat/github-provider
Closed

Feat/GitHub provider#2843
carvido1 wants to merge 13 commits intoexternal-secrets:mainfrom
carvido1:feat/github-provider

Conversation

@carvido1
Copy link
Copy Markdown

@carvido1 carvido1 commented Nov 3, 2023

Problem Statement

What is the problem you're trying to solve?
I'm implementing the github secret store provider

Related Issue

Implements part of #1727

Proposed Changes

How do you like to solve the issue and why?
I have implemented the start of the github provider for external secrets based on github actions organisations secrets.

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test
  • I ensured my PR is ready for review with make reviewable

Copy link
Copy Markdown
Contributor

@paul-the-alien paul-the-alien bot left a comment

Choose a reason for hiding this comment

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

Greetings!
Thank you for contributing to this project!
If this is your first time contributing, please make
sure to read the Developer and Contributing Process guides.
Please also mind and follow our Code of Conduct.

Useful commands:

  • make fmt: Formats the code
  • make check-diff: Ensures the branch is clean
  • make reviewable: Ensures a PR is ready for review

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@moolen moolen mentioned this pull request Dec 23, 2023
@JorgenSQ
Copy link
Copy Markdown

Any work being done on this, or is it stale?

We would LOVE a provider for github repository-secrets as we could omit all our custom scripts lifting this from github secrets, to azure keyvault today.

@shuheiktgw
Copy link
Copy Markdown
Contributor

Closing since the PR seems to be abandoned, and we also have #3115

@gusfcarvalho
Copy link
Copy Markdown
Member

gusfcarvalho commented Feb 7, 2024

@carvido1 are you still working on this? If so, let us know so we can reopen it. #3115 IMO covers a different feature

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.

4 participants