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

feat(Angular):updated to angular 17 #5983

Merged
merged 32 commits into from
Apr 8, 2024
Merged

feat(Angular):updated to angular 17 #5983

merged 32 commits into from
Apr 8, 2024

Conversation

dweinholz
Copy link
Contributor

Try to fulfill the following points before the Pull Request is merged:

  • Give a meaningfull description for the PR
  • The PR is reviewed by one of the team members.
  • If a linting PR exists, it must be merged before this PR is allowed to be merged.
  • It must be checked if anything in the Readme must be adjusted (development-, production-, setup).
  • It must be checked if any section in the wiki (https://cloud.denbi.de/wiki/) should be adjusted.
  • If the PR is merged in the master then a release should be be made.
  • The PR is responsive on smaller screens.
  • If the requirements.txt have changed, check if the patches still work
  • If the new code is readable, if not it should be well commented
  • In case the code is not well commented: An respectice commenting issue with tag "important" is opened.
  • If a squash of commits is required, it has been performed or will be performed at final merge
  • Finally a second team member checks if all requirements met

For releases only:

  • If the review of this PR is approved and the PR is followed by a release then the .env file
    in the cloud-portal repo should also be updated.

renovate bot and others added 6 commits November 14, 2023 16:29
@renovate
feat(Dependencies): Update all non-major dependencies
5b839ff 
| datasource | package                                | from    | to     |
| ---------- | -------------------------------------- | ------- | ------ |
| npm        | @angular-eslint/builder                | 16.2.0  | 16.3.1 |
| npm        | @angular-eslint/eslint-plugin          | 16.2.0  | 16.3.1 |
| npm        | @angular-eslint/eslint-plugin-template | 16.2.0  | 16.3.1 |
| npm        | @angular-eslint/schematics             | 16.2.0  | 16.3.1 |
| npm        | @angular-eslint/template-parser        | 16.2.0  | 16.3.1 |
| npm        | @coreui/coreui                         | 4.2.6   | 4.3.2  |
| npm        | @types/node                            | 20.8.10 | 20.9.0 |
| npm        | @typescript-eslint/eslint-plugin       | 6.10.0  | 6.11.0 |
| npm        | @typescript-eslint/parser              | 6.10.0  | 6.11.0 |
| npm        | eslint-plugin-jsdoc                    | 46.8.2  | 46.9.0 |
| npm        | ngx-cookie-service                     | 16.0.2  | 16.1.0 |
| npm        | prettier                               | 3.0.3   | 3.1.0  |
fix(Package): Adjust to bootstrap 5
167b526
adjusted new project form
e42f46f
table adjustments and leave project button
1dde9f6
@denbicloud
Branch was auto-updated.
d6c56a4
@dweinholz
feat(Angular):updated to angular 17
2833390
@dweinholz dweinholz requested a review from vktrrdk December 18, 2023 09:09
@dweinholz dweinholz removed the request for review from vktrrdk December 18, 2023 11:50
@dweinholz dweinholz requested a review from vktrrdk April 8, 2024 12:09
@dweinholz dweinholz merged commit 527951b into dev Apr 8, 2024
2 of 4 checks passed
@dweinholz dweinholz deleted the feat/angular_17 branch April 8, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vktrrdk vktrrdk Awaiting requested review from vktrrdk

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

Successfully merging this pull request may close these issues.
3 participants
@dweinholz @denbicloud @vktrrdk