Update compiler #725

Merged

Plume_migration_agent merged 7 commits from update-compiler into master

trinity-1686a commented

Owner

update the compiler, and fix deprecation warnings & assimilated

❤️ 2 🎉 2

trinity-1686a commented

Poster

Owner

hum, it's broken when using ructe 0.6.4, but we get deprecation warnings (hard errors due to -D all when using clippy) when using an older version (0.6.2) 😕

hum, it's broken when using ructe 0.6.4, but we get deprecation warnings (hard errors due to `-D all` when using clippy) when using an older version (0.6.2) :confused:

codecov[bot] commented (Migrated from github.com)

Codecov Report

Merging #725 into master will increase coverage by 0.02%.
The diff coverage is 34%.

@@            Coverage Diff             @@
##           master     #725      +/-   ##
==========================================
+ Coverage   39.12%   39.15%   +0.02%     
==========================================
  Files          73       73              
  Lines        9686     9650      -36     
  Branches     2315     2184     -131     
==========================================
- Hits         3790     3778      -12     
- Misses       4809     4820      +11     
+ Partials     1087     1052      -35

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/725?src=pr&el=h1) Report > Merging [#725](https://codecov.io/gh/Plume-org/Plume/pull/725?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/72464fb4283d252e345c7a416530384e60bc2d58?src=pr&el=desc) will **increase** coverage by `0.02%`. > The diff coverage is `34%`. ```diff @@ Coverage Diff @@ ## master #725 +/- ## ========================================== + Coverage 39.12% 39.15% +0.02% ========================================== Files 73 73 Lines 9686 9650 -36 Branches 2315 2184 -131 ========================================== - Hits 3790 3778 -12 - Misses 4809 4820 +11 + Partials 1087 1052 -35 ```

elegaanz (Migrated from github.com) approved these changes

elegaanz (Migrated from github.com) left a comment

If it compiles, it is OK I guess. I read the code quickly, and it seems good. LGTM! And thank you :)

kiwii referenced this issue from a commit

Update compiler (#725) * update compiler to recent nightly * cargo fmt * fix clippy but ructe code is still complaining * update circleci and rocket_csrf * fix last clippy problems * cargo fmt * fix build issue was caused be improper handling of recursive template (comment) by old ructe, which is fixed on newer version

Reviewers

The pull request has been merged as 3663bffe5c.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b update-compiler master

git pull origin update-compiler

Step 2:

Merge the changes and update on Forgejo.

git checkout master

git merge --no-ff update-compiler

git push origin master

No reviewers

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Loading…

There is no content yet.

Labels Milestones

Update compiler #725

Codecov Report

Reviewers

Step 1:

Step 2: