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

Add main README for jdt.debug and clean-up CONTRIBUTING file #532

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

HannesWell
Copy link
Contributor

What it does

Add main README for jdt.debug and clean-up CONTRIBUTING file.
The content of both is based on the content of the corresponding files in jdt.core/ui.

Author checklist

Comment on lines +3 to +11
JDT Debug implements Java debugging support and works with any JDPA-compliant target Java VM.
It is implemented on top of the language independent "debug model" provided by the platform debugger.
JDT debug provides the following debugging features:

- Launching of a Java VM in either run or debug mode
- Attaching to a running Java VM
- Expression evaluation in the context of a stack frame
- Scrapbook pages for interactive Java code snippet evaluation
- Dynamic class reloading where supported by Java virtual machine
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This section is mainly based on the content of
https://eclipse.dev/eclipse/debug/index.php

CONTRIBUTING Outdated
See https://github.com/eclipse-jdt/.github/blob/main/CONTRIBUTING.md
Copy link
Contributor

Choose a reason for hiding this comment

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

is that file needed at all? when its deleted it shout be automatically taken from .github

Copy link
Contributor

Choose a reason for hiding this comment

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

Removal will be a good choice.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is that file needed at all? when its deleted it shout be automatically taken from .github

I cannot say if there are legal requirements or alike and I also don't know if it's inherited. I just know that many, if not all, repos in the platform, equinox, PDE organization have it. Maybe just to avoid dead links?

But I removed it now.

Copy link
Contributor

Choose a reason for hiding this comment

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

It was a honest question. I know from https://docs.github.com/de/communities/setting-up-your-project-for-healthy-contributions/setting-guidelines-for-repository-contributors that https://github.com/eclipse-jdt/eclipse.jdt.ui/contribute does link it. But i do not know who would know how to go to that URL, i don't see a link to it anywhere and have never used it myself

For more information about JDT Debug refer to the [JDT Debug website](https://eclipse.dev/eclipse/debug/index.php).
For general information about JDT and important links, refer to the [JDT wiki page](https://github.com/eclipse-jdt/eclipse.jdt.core/wiki) or the [JDT project overview page](https://projects.eclipse.org/projects/eclipse.jdt).

## Contributing
Copy link
Contributor

Choose a reason for hiding this comment

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

does this file need a Contributing section? The existing of Contributing.md should be self explaining

Copy link
Contributor

@SougandhS SougandhS Oct 10, 2024

Choose a reason for hiding this comment

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

I agree with @jukzi on this. looks good without Contributing section
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

does this file need a Contributing section? The existing of Contributing.md should be self explaining

The contributing.md is now removed and even if it's inherited from the organization I think it's good to have a few words about contributions and to provide the link to the orga's contributing section explicitly.

Furthermore I wanted to add the Oomph setup button for the setup to be added for repository (see eclipse-platform/eclipse.platform.releng.aggregator#2430) in that section in a follow-up PR, similar to how it is e.g. in the platform repository:
https://github.com/eclipse-platform/eclipse.platform.ui/?tab=readme-ov-file#how-to-contribute

Therefore I would be in favor to keep this.

@jukzi
Copy link
Contributor

jukzi commented Oct 10, 2024

@SougandhS can you review, please?

@SougandhS
Copy link
Contributor

@SougandhS can you review, please?

Sure @jukzi

The content is based on the content of the corresponding files in
jdt.core/ui.
@jukzi jukzi merged commit 8be19a1 into eclipse-jdt:master Oct 10, 2024
7 of 8 checks passed
@jukzi
Copy link
Contributor

jukzi commented Oct 10, 2024

lets try

@jukzi
Copy link
Contributor

jukzi commented Oct 10, 2024

https://github.com/eclipse-jdt/eclipse.jdt.debug/contribute now shows the .github link :-)

@HannesWell
Copy link
Contributor Author

Thanks for submitting.

https://github.com/eclipse-jdt/eclipse.jdt.debug/contribute now shows the .github link :-)

That's nice, I didn't know this was possible.
But I don't know either, where this link is accessible from the UI?
But if it works well in general, we could delete many of the contributing files in many repositories that are more or less empty.

@HannesWell HannesWell deleted the add-readme branch October 10, 2024 20:49
@jukzi
Copy link
Contributor

jukzi commented Oct 10, 2024

we could delete many of the contributing files

yes, please. While Readme makes sense per repository, the governing rules belong to the organization and are same for all repos the org owns.

@HannesWell
Copy link
Contributor Author

we could delete many of the contributing files

yes, please. While Readme makes sense per repository, the governing rules belong to the organization and are same for all repos the org owns.

Created eclipse-platform/eclipse.platform.releng.aggregator#2448 to track that.

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