See #2647824: [policy and patch] Determine what should go in CHANGELOG.txt for 8.x core releases for the policy issue.

I don't know if this patch fully complies with that policy, but it's a stab at listing some of the important changes in 8.1.0.

CommentFileSizeAuthor
#13 2709569-13.patch1.62 KBdawehner
#8 interdiff.txt588 bytesxjm
#8 2456477-7.patch2.49 KBxjm
#6 interdiff-changelog.txt1.84 KBxjm
#6 2709569-5.patch2.36 KBxjm
8.1.0-changelog.patch2.11 KBeffulgentsia

Comments

effulgentsia created an issue. See original summary.

effulgentsia’s picture

effulgentsia commented

Note that the patch does not change (fill in the date for) the top line: Drupal 8.1.0, xxxx-xx-xx (development version). That's a separate step that's needed prior to tagging 8.1.0.

effulgentsia’s picture

effulgentsia commented 
+++ b/core/CHANGELOG.txt
@@ -7,6 +7,40 @@ Drupal 8.1.0, xxxx-xx-xx (development version)
+    * Simplified migrations from configuration entities to plugins.

I realize as I read this that it might be ambiguous. "migrations" refers to Migrate module migrations, not a generic concept of converting any config entity to a plugin :)

xjm’s picture

xjm
she/her
Primary language English
commented 
  1. +++ b/core/CHANGELOG.txt
@@ -7,6 +7,40 @@ Drupal 8.1.0, xxxx-xx-xx (development version)
       install to get dependencies.
     * See https://www.drupal.org/documentation/install/download#git
       for instructions.

    For those who might have wondered "Where's the composer change?" it's already there in HEAD. :) 

  2. +++ b/core/CHANGELOG.txt
@@ -7,6 +7,40 @@ Drupal 8.1.0, xxxx-xx-xx (development version)
+- Updated vendor libraries:
+    * Updated to Symfony 2.8.4.
+    * Updated to CKEditor 4.5.8.
+    * Updated to Modernizr 3.3.1.

    I think we don't necessarily need to include these -- we also update the patch versions in patch releases. (The section on the added libraries OTOH makes sense.) 

  3. +++ b/core/CHANGELOG.txt
@@ -7,6 +7,40 @@ Drupal 8.1.0, xxxx-xx-xx (development version)
+    * Simplified migrations from configuration entities to plugins.

    How about "Simplified Migrate API by replacing migration configuration entities with plugins." ? 

  4. +++ b/core/CHANGELOG.txt
@@ -7,6 +7,40 @@ Drupal 8.1.0, xxxx-xx-xx (development version)
+- Testing improvements:
+    * Added support for automated JavaScript testing.

    I wonder if the BTB and PHPUnit improvements are also worth a mention?

dawehner’s picture

dawehner
Primary language German
commented

I think we don't necessarily need to include these -- we also update the patch versions in patch releases. (The section on the added libraries OTOH makes sense.)

Given that we actually updated a minor version of symfony might be not unimportant.

I wonder if the BTB and PHPUnit improvements are also worth a mention?

We could add an entry that the old kernel test is really deprecated now for example.

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 2709569-5.patch2.36 KB
new interdiff-changelog.txt1.84 KB

Here's my proposed changes. I also changed the release date because this will be a commit immediately before release either way. We don't need to be coy. :)

Also RTBCing even though I made small changes since this is vastly better than nothing (and really pretty comprehensive).

xjm’s picture

xjm
she/her
Primary language English
commented

@dawehner: Oops, crossposted with you, sorry!

xjm’s picture

xjm
she/her
Primary language English
commented
StatusFileSize
new 2456477-7.patch2.49 KB
new interdiff.txt588 bytes

Adding back the library updates as @dawehner points out that Symfony 2.8 in particular is noteworthy.

I left off KTB since it is almost-but-not-quite done: #2456477: Convert deprecated \Drupal\simpletest\KernelTestBase tests to KernelTestBaseNG

dawehner’s picture

dawehner
Primary language German
commented

Well, I would try to explicitly point out the minor version update of symfony, and maybe link to some page, but its hard to find one.

  • catch committed 29ec084 on 8.2.x 
    Issue #2709569 by xjm, effulgentsia, dawehner: CHANGELOG.txt additions...

  • catch committed 01f39e3 on 8.1.x 
    Issue #2709569 by xjm, effulgentsia, dawehner: CHANGELOG.txt additions...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

dawehner’s picture

dawehner
Primary language German
commented
Status: Fixed » Needs review
StatusFileSize
new 2709569-13.patch1.62 KB

Just some small followup:

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Those improvements look good to me.

  • webchick committed c48884e on 8.2.x 
    Issue #2709569 follow-up by dawehner: Adjustments to CHANGELOG.txt

  • webchick committed e7b257f on 8.1.x 
    Issue #2709569 follow-up by dawehner: Adjustments to CHANGELOG.txt

(...
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed c48884e and pushed to 8.2.x, cherry-picked to 8.1.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.