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

fix(BREAKING): remove dead --prompt flag #22038

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

iuioiua
Copy link
Contributor

@iuioiua iuioiua commented Jan 22, 2024

It appears the --prompt flag has done nothing for some time. Perhaps, since #13650. Classifying this as a dead functionality removal for this reason.

Did this while working on #22021.

@dsherret
Copy link
Member

dsherret commented Jan 22, 2024

@iuioiua we probably shoudln't be categorizing most of these PRs as chores because then they won't show up in the release notes. Probably fix(BREAKING) would be more approppriate

@dsherret dsherret added this to the 2.0 milestone Jan 22, 2024
@iuioiua iuioiua changed the title chore: remove dead --prompt flag fix(BREAKING): remove dead --prompt flag Jan 22, 2024
@iuioiua
Copy link
Contributor Author

iuioiua commented Jan 22, 2024

Ah, ok. FYI, I talked to Bartek offline, and we both think this might be fine to have as part of the 1.40. This would only be breaking if someone hadn't updated since nearly 2 years ago when the behaviour was changed. What do we think?

@dsherret
Copy link
Member

Sure. Easy for them to fix

@dsherret dsherret modified the milestones: 2.0, 1.40 Jan 22, 2024
Copy link
Member

@littledivy littledivy left a comment

Choose a reason for hiding this comment

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

LGTM for 1.40

@bartlomieju bartlomieju merged commit d1eed98 into denoland:main Jan 23, 2024
14 checks passed
@iuioiua iuioiua deleted the remove-prompt-flag branch January 23, 2024 17:56
bartlomieju pushed a commit to bartlomieju/deno that referenced this pull request Jan 23, 2024
It appears the `--prompt` flag has done nothing for some time. Perhaps,
since denoland#13650. Classifying this as a dead functionality removal for this
reason.

Did this while working on denoland#22021.
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.

4 participants