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

Samwelkanda/python/kiota abstractions setup #925

Merged
merged 67 commits into from
Feb 2, 2022

Conversation

samwelkanda
Copy link
Contributor

@samwelkanda samwelkanda commented Dec 7, 2021

partial #367

@samwelkanda samwelkanda self-assigned this Dec 7, 2021
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 7, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

Can you add an entry into the changelog please?
As well as update the readme file (the table with the links)

@baywet baywet added this to the TypeWriter Replacement milestone Jan 24, 2022
@samwelkanda
Copy link
Contributor Author

Can you add an entry into the changelog please? As well as update the readme file (the table with the links)

Updated in dfc5583

@samwelkanda samwelkanda requested a review from baywet January 27, 2022 12:05
README.md Outdated
| TypeScript/JavaScript | [✔](https://github.com/microsoft/kiota/projects/2) | [✔](./abstractions/typescript) | [JSON](./serialization/typescript/json) | [Anonymous](./abstractions/typescript/src/authentication/anonymousAuthenticationProvider.ts), [Azure](./authentication/typescript/azure) | [✔](./http/typescript/fetch) | [link](https://microsoft.github.io/kiota/get-started/typescript) |
| Language | Generation | Abstractions | Serialization | Authentication | HTTP | Required tools & dependencies |
| -------- | ---------- | ------------ | ------------- | -------------- | ---- | -------------- |
| CSharp | [✔](https://github.com/microsoft/kiota/projects/5) | [✔](./abstractions/dotnet) | [JSON](./serialization/dotnet/json) | [Anonymous](./abstractions/dotnet/src/authentication/AnonymousAuthenticationProvider.cs), [Azure](./authentication/dotnet/azure) | [✔](./http/dotnet/httpclient) | [link](./docs/requiredtools/dotnet.md) |
Copy link
Member

Choose a reason for hiding this comment

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

Please revert that change and put back the links to the github.io site when available (dotnet, java, go, ruby, typescript)

Copy link
Member

Choose a reason for hiding this comment

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

but just to be clear, this page still needs to be updated for the Python row

@baywet
Copy link
Member

baywet commented Jan 27, 2022

also another things that's missing in that batch of PRs is a github actions worklfow to build the library.

calebkiage
calebkiage previously approved these changes Feb 1, 2022
run: flit publish
working-directory: ${{ env.relativePath }}
env:
FLIT_INDEX_URL: https://upload.pypi.org/legacy/
Copy link
Member

Choose a reason for hiding this comment

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

is this the public feed?

Copy link
Contributor Author

@samwelkanda samwelkanda Feb 1, 2022

Choose a reason for hiding this comment

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

Yes, that is the link to public index. It just happens to be named "legacy".

Copy link
Member

Choose a reason for hiding this comment

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

I don't think we should publish to the public index just yet in case we need to make breaking changes. Github packages doesn't seem to support Python.
May I suggest we create a new a ADO package feed and use that instead for the time being? https://microsoftgraph.visualstudio.com/Graph%20Developer%20Experiences/_packaging?_a=feed&feed=PowerShellSDK_BuildFeed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok. What if we publish to the test Index https://test.pypi.org for now then?

Copy link
Member

Choose a reason for hiding this comment

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

this one is public I assume? but it's understood by the community that things on this index are not meant for production, correct? and can we pull a package entirely from that index? (I'm thinking of the case where we change the name of the package).
If the answer to both question is yes, please proceed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, that is the case.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for making the change! Let me know if you need tme to add secrets to the repo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will. I currently don't have the privileges to add the secrets myself (can't access the project settings).

@samwelkanda samwelkanda force-pushed the samwelkanda/python/kiota-abstractions-setup branch from 3ff66d6 to bb24e21 Compare February 1, 2022 21:54
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 1, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@samwelkanda samwelkanda requested a review from baywet February 1, 2022 21:58
@samwelkanda samwelkanda merged commit cd7d63c into main Feb 2, 2022
@samwelkanda samwelkanda deleted the samwelkanda/python/kiota-abstractions-setup branch February 2, 2022 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants