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

docs: design of adding config to support additional directories #123

Merged
merged 5 commits into from
Dec 20, 2022

Conversation

ningziwen
Copy link
Member

Signed-off-by: Ziwen Ning ningziwe@amazon.com

Issue #, if available:
#107

Description of changes:
Doc

Testing done:

  • [ X ] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
@ningziwen ningziwen changed the title docs: add design doc: config to support additional workspace directories docs: design of adding config to support additional workspace directories Dec 20, 2022
@ningziwen ningziwen changed the title docs: design of adding config to support additional workspace directories docs: design of adding config to support additional directories Dec 20, 2022
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
pendo324
pendo324 previously approved these changes Dec 20, 2022
mharwani
mharwani previously approved these changes Dec 20, 2022
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
@ningziwen ningziwen dismissed stale reviews from mharwani and pendo324 via 26f8925 December 20, 2022 19:13
@ningziwen ningziwen merged commit f2a6346 into runfinch:main Dec 20, 2022
@ningziwen ningziwen deleted the dir-doc branch December 20, 2022 19:17
ningziwen added a commit that referenced this pull request Dec 26, 2022
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>

Issue #, if available:
#107

*Description of changes:*
add config to support additional directories according to the doc added
in #123

*Testing done:*



- [ X ] I've reviewed the guidance in CONTRIBUTING.md


#### License Acceptance

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
ahsan-z-khan pushed a commit to ahsan-z-khan/finch that referenced this pull request Jan 11, 2023
…inch#123)

Signed-off-by: Ziwen Ning <ningziwe@amazon.com>

Issue #, if available:
runfinch#107

*Description of changes:*
Doc

*Testing done:*



- [ X ] I've reviewed the guidance in CONTRIBUTING.md


#### License Acceptance

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
ahsan-z-khan pushed a commit to ahsan-z-khan/finch that referenced this pull request Jan 11, 2023
Signed-off-by: Ziwen Ning <ningziwe@amazon.com>

Issue #, if available:
runfinch#107

*Description of changes:*
add config to support additional directories according to the doc added
in runfinch#123

*Testing done:*



- [ X ] I've reviewed the guidance in CONTRIBUTING.md


#### License Acceptance

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

Signed-off-by: Ziwen Ning <ningziwe@amazon.com>
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.

7 participants