incubator-answer icon indicating copy to clipboard operation
incubator-answer copied to clipboard

Master Update

Open Justman100 opened this issue 5 months ago • 4 comments

Languages

  • Pronunciation optimised

Frontend

  • Use NodeJS 18 and higher
  • Removed pnpm force
  • Updated all NodeJS dependencies for the frontend to latest version
  • No link to the GitHub release page of Answer, if version already up to date
  • Version check is skipped if Answer was installed directly from GitHub

Backend

  • Allowing of uppercase usernames
  • Allowing of shorter/longer (user)names, passwords, mails and site names
  • Allowing of shorter/longer object ids
  • Allowing of shorter/longer new/old revision ids
  • Allowing of shorter/longer querys
  • Allowing of shorter/longer plugin slug names
  • Allowing of shorter/longer report contents
  • Allowing of shorter/longer Question titles, contents and ids

Justman100 avatar Feb 12 '24 13:02 Justman100

First of all, I want to talk about how to make a better contribution. From a developer's point of view, each PR should be clear. The current PR solves a specific problem or completes a complete function; if you If there are multiple features or Isuu that need to be contributed, they should be split into multiple PRs to submit, so that there will be no additional burden when others come to Reiview.

Secondly, when changing some project configurations or basic environments, a discussion needs to be initiated first, and then community users will decide whether these configurations or environments need to be changed.

Use NodeJS 18 and higher

We only need the minimum version to ensure the normal operation of the project

Removed pnpm force

We started with pnpm for a few reasons

  1. pnpm has better performance and memory management than yarn or npm.
  2. our CI also uses pnpm and our team uses pnpm as the front-end package management tool.
  3. for developers or contributors, it is up to them to decide which package management tool to use. They just need to make sure not to submit these files when submitting a PR.

updated all NodeJS dependencies for the frontend to latest version

whether dependencies need to be upgraded, we will decide based on the security report and whether the existing dependencies meet the functions of the current version. Each dependency upgrade requires a lot of testing, so a certain dependency needs to be upgraded. Please explain the reason for the upgrade, and we will update it. Determine whether to upgrade based on specific circumstances

No link to the GitHub release page of Answer, if version already up to date

If need to do this, please go to ISSU to initiate a discussion. We will decide whether adjustments are needed based on user feedback.

Version check is skipped if Answer was installed directly from GitHub

Reference here #764

Finally, thank you very much for your contribution. If you have any questions, you can continue the discussion.

shuashuai avatar Feb 27 '24 02:02 shuashuai

First of all, I want to talk about how to make a better contribution. From a developer's point of view, each PR should be clear. The current PR solves a specific problem or completes a complete function; if you If there are multiple features or Isuu that need to be contributed, they should be split into multiple PRs to submit, so that there will be no additional burden when others come to Reiview.

Secondly, when changing some project configurations or basic environments, a discussion needs to be initiated first, and then community users will decide whether these configurations or environments need to be changed.

Use NodeJS 18 and higher

We only need the minimum version to ensure the normal operation of the project

Removed pnpm force

We started with pnpm for a few reasons

1. pnpm has better performance and memory management than yarn or npm.

2. our CI also uses pnpm and our team uses pnpm as the front-end package management tool.

3. for developers or contributors, it is up to them to decide which package management tool to use. They just need to make sure not to submit these files when submitting a PR.

updated all NodeJS dependencies for the frontend to latest version

whether dependencies need to be upgraded, we will decide based on the security report and whether the existing dependencies meet the functions of the current version. Each dependency upgrade requires a lot of testing, so a certain dependency needs to be upgraded. Please explain the reason for the upgrade, and we will update it. Determine whether to upgrade based on specific circumstances

No link to the GitHub release page of Answer, if version already up to date

If need to do this, please go to ISSU to initiate a discussion. We will decide whether adjustments are needed based on user feedback.

Version check is skipped if Answer was installed directly from GitHub

Reference here #764

Finally, thank you very much for your contribution. If you have any questions, you can continue the discussion.

  1. So, one cannot use capital letters during the installation, which is definitely not the way it should be! The database doesn't care if the username is uppercase or lowercase!
  2. Updating dependencies only when it's needed is the stupidest setting you can have! Nobody updates their dependencies only when it's needed! One always does it, even if there are no security risks
  3. In the script to build the plugins, the sleep statements are intentional, because one can only read the text that is output by echo before the script bombs the console, and one can no longer read it properly
  4. Do I have to initialise the TryToInstallByEnv again so that the variableinstallByEnv gets a value?
  5. The tag check when asking questions is superfluous, as one does not need it to be able to ask questions (a tag force is also quite bold)

Furthermore, I test all my changes before I push them! So all my commits can be merged without hesitation

Justman100 avatar Mar 15 '24 06:03 Justman100

And if I were to discuss all my changes that I want to implement first, I could just forget about it, because everyone else could then simply commit these changes themselves because they would then know about them

Justman100 avatar Mar 15 '24 07:03 Justman100

@shuashuai @LinkinStars

Justman100 avatar Mar 26 '24 11:03 Justman100

Please split according to the improvement suggestions and resubmit! The current PR was discussed as not meeting the requirements.

shuashuai avatar Apr 16 '24 01:04 shuashuai