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.

Sign up for GitHub

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

Jump to bottom

Update Docu Link zu v3.0.x #616

Merged
merged 1 commit into from
Jan 28, 2025
Merged

Update Docu Link zu v3.0.x #616

merged 1 commit into from
Jan 28, 2025

Conversation

JohannMaierhofer
Copy link

No description provided.

tobidope
tobidope approved these changes Jan 28, 2025
View reviewed changes
Copy link
Member

@tobidope tobidope left a comment

Choose a reason for hiding this comment

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

Wäre es sinnvoll das in eine Konfiguration zu übertragen?

@JohannMaierhofer JohannMaierhofer merged commit 2cf0adf into openjverein:master Jan 28, 2025
2 checks passed
@JohannMaierhofer JohannMaierhofer deleted the docu_update branch January 28, 2025 09:53
@JohannMaierhofer
Copy link
Author

Wäre es sinnvoll das in eine Konfiguration zu übertragen?

Wie würde das gehen? Die Version ist in der URL der Help. Sie muss in den static Strings in dieser Datei enthalten sein.

Aber wenn du einen Vorschlag hast kann man das machen.

@dippeal
Copy link
Member

dippeal commented Jan 28, 2025

Beim build die version aus plugin.xml auslesen, als Präfix das "v" anfügen und die Patch Stelle durch ein "x" ersetzen?
Übrigens müsste in der plugin.xml dann noch die Version auf 3.0.0 eingestellt werden. Könnte man jetzt machen oder wenn die notwendigen 3.0 PRs abgeschlossen sind.

@JohannMaierhofer
Copy link
Author

Beim build die version aus plugin.xml auslesen, als Präfix das "v" anfügen und die Patch Stelle durch ein "x" ersetzen? Übrigens müsste in der plugin.xml dann noch die Version auf 3.0.0 eingestellt werden. Könnte man jetzt machen oder wenn die notwendigen 3.0 PRs abgeschlossen sind.

Das ist etwas was ich nicht anlange weil ich mich mit den Scripts nicht auskenne. Wenn du das machen möchtest, kannst du das gerne tun.

@tobidope
Copy link
Member

Wenn ihr meinen pr #614 approved, könnten wir auf Maven umstellen und solche Sachen mit dem großen plugin Zauber von maven lösen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobidope tobidope tobidope approved these changes

@mbmueller mbmueller mbmueller approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JohannMaierhofer @dippeal @tobidope @mbmueller