Skip to content

Conversation

@SableRaf
Copy link
Collaborator

@SableRafSableRaf commented Mar 20, 2025

skip ci

closes#985

@SableRafSableRaf requested a review from SteftervMarch 20, 2025 13:01
@SableRafSableRaf mentioned this pull request Mar 20, 2025
BUILD.md Outdated
### Running Processing

The main task to run or debug the PDE is `app:run` this run the application with `compose desktop`
The main task to run or debug the PDE is `run`
Copy link
Contributor

Choose a reason for hiding this comment

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

Period (.) missing at end of sentence.

Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

Good catch! Fixed.

Copy link
Contributor

@babaissarkarbabaissarkarMar 20, 2025

Choose a reason for hiding this comment

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

I'd also suggest mentioning the exact command. Not everyone who's building from source may understand what a gradle task means (aka there are people who may be interested in building from source solely because it gives them access to latest features).

The main task to run or debug the PDE is `run`. That means you just need to run `./gradlew run` (Linux) or `./gradlew.bat run` (Windows) to build and run Processing.

Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

Done! I'll let @Stefterv review the changes.

@SteftervStefterv marked this pull request as ready for review March 20, 2025 18:44
@SteftervStefterv added this to the 4.4.1 milestone Mar 20, 2025
@SableRafSableRaf merged commit f168214 into mainMar 20, 2025
@SteftervStefterv deleted the SableRaf-patch-1 branch March 25, 2025 12:23
Sign up for freeto 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.

Can't build latest checkout

4 participants

@SableRaf@Stefterv@babaissarkar