This repository has been archived by the owner on Jul 3, 2023. It is now read-only.
Open Source #8
Merged
Open Source #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8 tasks
See https://github.com/cla-assistant/github-action Instead of manually having to manage this, we can have a bot do it for us. This will mean that people who contribute will agree and then this will store this in a file in this repository. Note, if we don't want Stitch Fix people to be hit by this, we just need to add them to the allow list.
It was decided that this would be easier for us to open source and manage with an Apache 2.0 license.
|
This is so exciting! |
|
i think you've scrubbed well and I'm not anxious about anything that's still in the markdowns, though, it may be worth adding a comment to the readme somewhere that states all of the code examples are just for illustration and do not reflect anything we are doing internally. just to cover ourselves. |
because to someone not from Stitch Fix it wouldn't be clear what is meant.
Due to feedback.
Due to conversation with legal, they think it's best to go with this license.
They were all running together on one line.
skrawcz
commented
Sep 2, 2021
This was referenced Feb 23, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR does the following:
I'll squash merge this PR.