Skip to content

Comments

add --skip-existing to publish#2812

Merged
radoering merged 2 commits intopython-poetry:masterfrom
davidszotten:publish_skip_existing
Mar 29, 2022
Merged

add --skip-existing to publish#2812
radoering merged 2 commits intopython-poetry:masterfrom
davidszotten:publish_skip_existing

Conversation

@davidszotten
Copy link
Contributor

@davidszotten davidszotten commented Aug 14, 2020 •
edited
Loading

Pull Request Check List

Resolves: #2254

WIP to check approach.

  • Added tests for changed code.
  • Updated documentation for changed code.

Questions

  • does this seem reasonable?
  • twine is apache 2 licensed, is copying their impl ok?
  • should we consider bundling together the "options" (dry_run and skip_existing) into an options object or similar to have fewer things to thread through all the functions?

/cc @abn

@davidszotten davidszotten force-pushed the publish_skip_existing branch 3 times, most recently from 86d7998 to a530706 Compare August 17, 2020 13:30
@abn abn marked this pull request as draft August 30, 2020 13:40
@abn abn changed the base branch from develop to master August 30, 2020 13:40
@davidszotten
Copy link
Contributor Author

davidszotten commented Apr 28, 2021

hey @abn . did you mark this as wip/draft? what is the next step here?

@davidszotten davidszotten force-pushed the publish_skip_existing branch from a530706 to afb0060 Compare April 28, 2021 11:07
@davidszotten davidszotten marked this pull request as ready for review April 28, 2021 12:14
@davidszotten davidszotten force-pushed the publish_skip_existing branch from afb0060 to 383e3cc Compare June 17, 2021 10:43
@davidszotten davidszotten force-pushed the publish_skip_existing branch from 383e3cc to 0a1c0f2 Compare July 9, 2021 15:47
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 9, 2021

SonarCloud Quality Gate failed.

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

No Coverage information No Coverage information
5.9% 5.9% Duplication

@davidszotten
Copy link
Contributor Author

davidszotten commented Jul 9, 2021

@sdispater what's the policy for the sonarcloud report? apparently i'm adding a "code smell" though i'm just trying to be consistent with the surrounding code

@davidszotten davidszotten changed the title [WIP] add --skip-existing to publish add --skip-existing to publish Jul 14, 2021
IsmaelMartinez
IsmaelMartinez reviewed Jul 15, 2021
View reviewed changes
@davidszotten davidszotten force-pushed the publish_skip_existing branch 2 times, most recently from 121ce51 to d66395f Compare July 22, 2021 09:49
@cob16
Copy link

cob16 commented Aug 27, 2021

How is this PR going? Is this ready to merge now?

@IsmaelMartinez
Copy link

IsmaelMartinez commented Aug 27, 2021

Someone from the project needs to review it. LGTM

@IsmaelMartinez
Copy link

IsmaelMartinez commented Sep 3, 2021

@sdispater or @abn ? do you think you can assign someone to review this and/or review it so we can get this feature? Much appreciated and thanks a lot for this awesome project.

@czomo
Copy link

czomo commented Mar 24, 2022

Any chance this will be reviewed in near future?

@davidszotten davidszotten force-pushed the publish_skip_existing branch from d66395f to 8a75e9a Compare March 24, 2022 19:01
@czomo
Copy link

czomo commented Mar 26, 2022

@finswimmer and @radoering could you help us here?

radoering
radoering requested changes Mar 26, 2022
View reviewed changes
Copy link
Member

@radoering radoering left a comment

Choose a reason for hiding this comment

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

At first, thanks for your contribution and your patience. In my opinion, this option is useful. (I even had the use case myself once.)

Regarding the license question, I think that is difficult to answer. APACHE license allows using the code, but demands to include the license notice. In order to avoid that, I'd prefer to rewrite the code. Since the function is not that big, there are not so many options. (It even may not be necessary because it's so small. I'm not a licensing expert.)

Anyway, I made some suggestions. Feel free to adopt or implement your own ideas.

I just reviewed the _ignore_existing function for now and will continue later.

radoering
radoering requested changes Mar 26, 2022
View reviewed changes
@davidszotten davidszotten force-pushed the publish_skip_existing branch from 8a75e9a to 7e32ad3 Compare March 26, 2022 08:23
@davidszotten davidszotten requested a review from radoering March 27, 2022 12:18
radoering
radoering requested changes Mar 27, 2022
View reviewed changes
@davidszotten davidszotten force-pushed the publish_skip_existing branch from 7e32ad3 to ea2cba1 Compare March 27, 2022 18:53
@davidszotten davidszotten requested a review from radoering March 27, 2022 18:54
radoering
radoering requested changes Mar 28, 2022
View reviewed changes
@davidszotten
add --skip-existing to publish
d821a43 
add `--skip-existing` option to publish command to continue if an upload
fails due to the file already existing. this is helpful in e.g. ci
settings. resolves python-poetry#2254
@davidszotten davidszotten force-pushed the publish_skip_existing branch from ea2cba1 to d821a43 Compare March 28, 2022 18:51
@radoering radoering merged commit cf8999f into python-poetry:master Mar 29, 2022
@davidszotten
Copy link
Contributor Author

davidszotten commented Mar 29, 2022

thank you for all your help @radoering !

davidszotten added a commit to davidszotten/poetry that referenced this pull request Mar 29, 2022
@davidszotten
add changelog for python-poetry#2812
7bf190a
@davidszotten davidszotten mentioned this pull request Mar 29, 2022
Closed
@kasteph kasteph mentioned this pull request May 30, 2022
Merged
@radoering radoering mentioned this pull request Aug 12, 2022
Closed
2 tasks
@github-actions
Copy link

github-actions bot commented Feb 29, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@radoering radoering radoering approved these changes

+1 more reviewer

@IsmaelMartinez IsmaelMartinez IsmaelMartinez left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add --skip-existing option to poetry publish

5 participants

@davidszotten @cob16 @IsmaelMartinez @czomo @radoering