Skip to content

Merge sources into one repo - #329

Merged
Andrew Omondi (andrueastman) merged 42 commits into
mainfrom
feature/kiota-python
Oct 14, 2024
Merged

Merge sources into one repo#329
Andrew Omondi (andrueastman) merged 42 commits into
mainfrom
feature/kiota-python

Conversation

@andrueastman

@andrueastmanAndrew Omondi (andrueastman) commented Sep 10, 2024

Copy link
Copy Markdown
Contributor

@baywetVincent Biret (baywet) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

started reviewing the PR but stopped midway through since I realized #331 still needs to be merged in this branch anyway

Comment threadpackages/authentication/azure/README.md Outdated
Comment threadpackages/http/httpx/README.md Outdated
@andrueastman

Copy link
Copy Markdown
ContributorAuthor

started reviewing the PR but stopped midway through since I realized #331 still needs to be merged in this branch anyway

Thanks Vincent Biret (@baywet), Just to set expectations here, once we close #331, I'll need to make some updates to do the following before this is fully ready for review.

  • Cleanup readmes and related docs
  • Validate workflows
  • Align versioning

@andrueastmanAndrew Omondi (andrueastman) changed the title WIP: Merge repos into one repoMerge sources into one repoOct 9, 2024
@andrueastman
Andrew Omondi (andrueastman) marked this pull request as ready for review October 9, 2024 10:35
@baywet

Copy link
Copy Markdown
Member

Andrew Omondi (@andrueastman) the policy definition probably needs to be updated here

@andrueastman

Copy link
Copy Markdown
ContributorAuthor

agreed.
I'll open a sepate PR targeting main in order to close #327 before we merge this.

@musaleMusale Martin (musale) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@baywet

Copy link
Copy Markdown
Member

this PR also needs to be ported microsoft/kiota-http-python#434

@andrueastman
Andrew Omondi (andrueastman) dismissed stale reviews from Musale Martin (musale) and Vincent Biret (baywet)October 14, 2024 08:59

The merge-base changed after approval.

@andrueastman

Copy link
Copy Markdown
ContributorAuthor

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

@github-actions

Copy link
Copy Markdown
Contributor

Conflicts have been resolved. A maintainer will take a look shortly.

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate FailedQuality Gate failed

Failed conditions
12 Security Hotspots
17.1% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@andrueastman

Copy link
Copy Markdown
ContributorAuthor

This should be good to go with the resolution of #351 and #350

Sign up for freeto 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.

Group python Kiota repositories update CI to create tags on releases based off the changelog

3 participants

@andrueastman@baywet@musale