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

remove demo requirement for release #227

Merged
merged 2 commits into from
Aug 18, 2020

Conversation

car-roll
Copy link
Collaborator

Demo is not using the latest release tags. Does not seem to be critical for releasing plugin

Copy link
Member

@dwnusbaum dwnusbaum left a comment

Choose a reason for hiding this comment

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

If you do this I would add a prominent notice to the top of demo/README.md that it is outdated and kept only as a reference, or maybe go ahead and delete the demo directory entirely and add a link to the top-level README that says something like "There is an outdated demo of this functionality here", and have it link to the demo folder as of tag docker-workflow-1.12 which seems to be the last time it was updated.

@car-roll car-roll merged commit d38c4b3 into jenkinsci:master Aug 18, 2020
@car-roll car-roll deleted the readme-cleanup branch August 18, 2020 17:03
@jglick
Copy link
Member

jglick commented Aug 18, 2020

Wait why are you deleting the demo?

@car-roll
Copy link
Collaborator Author

@jglick It appeared to be stuck at version 1.12 and there were some issues I had in getting it working with the latest snapshot. If you prefer, I can add it back in and just add in a note mentioning that it's on 1.12 in the README

@jglick
Copy link
Member

jglick commented Aug 18, 2020

there were some issues I had in getting it working with the latest snapshot

This is what I was missing. Up to you I guess. While maintaining this plugin I kept the demo running as a useful way to see the functionality working end-to-end in a semirealistic context. (Some artificialities around using docker.sock but using a real authenticated registry, etc.)

@jglick
Copy link
Member

jglick commented Aug 18, 2020

add it back in and just add in a note mentioning that it's on 1.12

No, that would be useless. My request would have been to fix the demo, unless there something fundamentally broken that could not be resolved without a lot of work.

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.

3 participants