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

Run As Context Menu shows Ant Run Configurations multiple times #527 #730

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SudhindraKKulkarni
Copy link

AntLaunchShortcutWithDialog now implements only ILaunchShortcut interface.
The common implementation of AntLaunchShortcut and AntLaunchShortcutWithDialog are moved to AntLaunchShortcutCommon class

Fixes #527

…se-platform#527

AntLaunchShortcutWithDialog now implements only ILaunchShortcut
interface.
The common implementation of AntLaunchShortcut and
AntLaunchShortcutWithDialog are moved to AntLaunchShortcutCommon class

Fixes eclipse-platform#527
Copy link
Member

@SarikaSinha SarikaSinha left a comment

Choose a reason for hiding this comment

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

Even after these changes, I see the duplicate entries.

@iloveeclipse
Copy link
Member

@SudhindraKKulkarni : please don't use "merge", we do not allow merge commits. Please rebase your original change on top of master & force push.

@jukzi
Copy link
Contributor

jukzi commented Aug 15, 2024

Please update the commit message to describe the fix not the problem. i.e. use "fix ..." or "... a single time"

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.

"Run As" Context Menu shows Ant Run Configurations multiple times
4 participants