Skip to content

[4.0] Codemirror update#27868

Merged
rdeutz merged 1 commit intojoomla:4.0-devfrom
brianteeman:cm
Feb 10, 2020
Merged

[4.0] Codemirror update#27868
rdeutz merged 1 commit intojoomla:4.0-devfrom
brianteeman:cm

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

Codemirror was locked at a specific version that is now very old.

This PR changes the setting in package.json so that it follows semver

Codemirror has always been very good at following semver so this should be totally safe.

(the update process also updated tinymce at the same time)

Codemirror was locked at a specific version that is now very old.

This PR changes the setting in package.json so that it follows semver

Codemirror has always been very good at following semver so this should be totally safe.

(the update process also updated tinymce at the same time)
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Feb 9, 2020
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Feb 9, 2020

I have tested this item ✅ successfully on 6bfb5d2


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27868.

1 similar comment
@richard67
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 6bfb5d2


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27868.

@richard67 richard67 removed NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Feb 9, 2020
@richard67
Copy link
Copy Markdown
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27868.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 9, 2020
@richard67 richard67 added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Feb 9, 2020
@richard67
Copy link
Copy Markdown
Member

@wilsonge @brianteeman I just see some "resolved" links in file package-lock.json have changed from https:// to http://. Is this ok or should these changes be reverted?

@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Feb 9, 2020

I think that's somehow based on OS or something it seems to change all the time. It's not ideal but it's not the end of the world either

@rdeutz rdeutz added this to the Joomla 4.0 milestone Feb 10, 2020
@rdeutz rdeutz merged commit 69052ca into joomla:4.0-dev Feb 10, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Feb 10, 2020
@brianteeman
Copy link
Copy Markdown
Contributor Author

Thnks

@brianteeman brianteeman deleted the cm branch February 10, 2020 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants