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

Open sourcing requirements #126

Merged
merged 20 commits into from
May 27, 2024
Merged

Open sourcing requirements #126

merged 20 commits into from
May 27, 2024

Conversation

truemiller
Copy link
Collaborator

@truemiller truemiller commented May 24, 2024

  • Updates to README
  • Improved release workflow; stops releases from other branches/pull requests (security risk)
  • Added Gitleaks workflow + initial .gitleaksignore
  • Removed private repo settings, and read-only update key

@truemiller truemiller changed the title (WIP): Open sourcing documentation requirements (WIP): Open sourcing requirements May 24, 2024
@truemiller truemiller requested review from Tanya-atatakai and mohandast52 and removed request for Tanya-atatakai May 24, 2024 17:46
@truemiller truemiller changed the title (WIP): Open sourcing requirements Open sourcing requirements May 24, 2024
Copy link
Collaborator

@Tanya-atatakai Tanya-atatakai left a comment

Choose a reason for hiding this comment

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

lgtm! a small question:
I noticed that poetry.lock was generated with "Poetry 1.8.2" but in the readme we say that we use "Poetry (1.7.1)" - @truemiller do you know which is correct?

Also noticed that this job is failing, do you what's the reason and if this can be fixed?
image

@truemiller
Copy link
Collaborator Author

lgtm! a small question: I noticed that poetry.lock was generated with "Poetry 1.8.2" but in the readme we say that we use "Poetry (1.7.1)" - @truemiller do you know which is correct?

Also noticed that this job is failing, do you what's the reason and if this can be fixed? image

Will update poetry version, and reach out to Viraj on this as it's a backend linter I believe

@truemiller truemiller merged commit 99e894e into main May 27, 2024
3 of 4 checks passed
@truemiller truemiller deleted the josh/open-source-reqs branch July 26, 2024 17:40
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