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

nit: remove my-custom-image default value from image build #6749

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

cdrage
Copy link
Contributor

@cdrage cdrage commented Apr 10, 2024

nit: remove my-custom-image default value from image build

What does this PR do?

Removes the default name which instead should be the placeholder that
shows you what information you can enter

Screenshot / video of UI

Screenshot 2024-04-10 at 4 16 18 PM

What issues does this PR fix or reference?

Closes #6748

How to test this PR?

  • Tests are covering the bug fix or the new feature

Also view the Build Page

Signed-off-by: Charlie Drage charlie@charliedrage.com

### What does this PR do?

Removes the default name which instead should be the placeholder that
shows you what information you can enter

### Screenshot / video of UI

<!-- If this PR is changing UI, please include
screenshots or screencasts showing the difference -->

### What issues does this PR fix or reference?

<!-- Include any related issues from Podman Desktop
repository (or from another issue tracker). -->

Closes containers#6748

### How to test this PR?

<!-- Please explain steps to verify the functionality,
do not forget to provide unit/component tests -->

- [X] Tests are covering the bug fix or the new feature

Also view the Build Page

Signed-off-by: Charlie Drage <charlie@charliedrage.com>
@cdrage cdrage requested review from benoitf and a team as code owners April 10, 2024 20:15
@cdrage cdrage requested review from dgolovin and jeffmaury and removed request for a team April 10, 2024 20:15
Copy link
Contributor

@dgolovin dgolovin left a comment

Choose a reason for hiding this comment

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

LGTM

@benoitf benoitf merged commit e4d4208 into containers:main Apr 11, 2024
8 checks passed
@podman-desktop-bot podman-desktop-bot added this to the 1.10.0 milestone Apr 11, 2024
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.

Remove 'my-custom-image' placeholder for build image.
4 participants