Skip to content

Comments

Changes required before we open the repo to the public#61

Merged
asullivan-blze merged 2 commits intomainfrom
release-prep
Feb 15, 2024
Merged

Changes required before we open the repo to the public#61
asullivan-blze merged 2 commits intomainfrom
release-prep

Conversation

@metadaddy
Copy link
Member

What and why?

Added LICENSE file, updated CONTRIBUTING.md and requirements.txt as required by Backblaze Legal department in preparation for opening the repo to the public.

How was this tested?

pip uninstall ansible
make test

No errors reported.

Checklist

  • Have you updated the VERSION file (if applicable)?
    No - we have to decide if/how we'll bump the version number for release.

@metadaddy
Copy link
Member Author

I'm guessing the test failure is due to a change in pyright's behavior, since the only change between the last two tests was the copyright year!

@asullivan-blze
Copy link
Contributor

This PR is blocked on #62 being merged prior to this one (needs a review to enable merge), and after this PR will need to be rebased against the main branch.

asullivan-blze added a commit that referenced this pull request Feb 15, 2024
* typechecking, readme (cleanup): update installation guidance; silence another pyright issue

Relates to PR #61.

* black (formatting): run latest black against the codebase
Copy link
Contributor

@asullivan-blze asullivan-blze left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

✨ 🍰 ✨

@asullivan-blze asullivan-blze merged commit 640343c into main Feb 15, 2024
@asullivan-blze asullivan-blze deleted the release-prep branch February 15, 2024 20:55
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.

2 participants