-
Notifications
You must be signed in to change notification settings - Fork 49
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
Use local abstractions sources instead of packages #92
Use local abstractions sources instead of packages #92
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From the perspective of getting these repos merged with the fewest changes, this is really good. However I'd really like to see the solutions properly reconciled into one, rather than this nesting. Whether that should be part of this or a follow up pass instead is another question.
… `main/HEAD` @ 63bac1681. * Copy abstractions sources * Adopt the abstractions migration * Merge misc files Signed-off-by: Yury-Fridlyand <[email protected]>
e1a5fa5
to
6d39fde
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we please cleanup the README in abstractions/ before merging?
| Yury Fridlyand | [Yury-Fridlyand](https://github.com/Yury-Fridlyand) | Bit Quill | | ||
| Max Ksyunz | [MaxKsyunz](https://github.com/MaxKsyunz) | Bit Quill | | ||
| Forest Vey | [forestmvey](https://github.com/forestmvey) | Bit Quill | | ||
| Mitchell Gale | [MitchellGale-BitQuill](https://github.com/MitchellGale-BitQuill) | Bit Quill | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So we're merging maintainers? :) This can be cleaned up later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are merging everything!
abstractions/README.md
Outdated
* `MAJOR.MINOR.PATH` where `MAJOR` is still supported as defined by the EOL policy of OpenSearch. | ||
* Note if the version exists but is not yet released it will resolve as a build candidate | ||
|
||
## Project Resources |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete all the non-abstraction parts. Things like COC are all at the root, we can't/shouldn't have duplicates in various parts of the repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated, thanks!
Signed-off-by: MaxKsyunz <[email protected]>
…ions-no-build Integrate abstractions projects into client build.
98fe5f9
Signed-off-by: Yury-Fridlyand [email protected]
Description
Abstractions lib sources were copied into .net client to use in tests. No need any more to store local nuget packages of abstractions, release them or worry about outdated/broken references to them.
Few fixes were made to let workflows pass. I have also update license headers to comply with #69/#78.
See Bit-Quill#40 for more details.
This PR follows option 2 in #86.
Issues Resolved
#86
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.