Problem/Motivation

With recent versions of composer, uppercase characters in package names is not tolerated.

[Composer\Repository\InvalidRepositoryException]
A repository of type "package" contains an invalid package definition: Invalid package information:
name : inorganik/countUp.js is invalid, it should not contain uppercase characters. We suggest using inorganik/count-up.js instead.

Invalid package definition:
{"name":"inorganik\/countUp.js","version":"2.0.6","type":"drupal-library","dist":{"url":"https:\/\/github.com\/inorganik\/countUp.js\/archive\/2.0.6.zip","type":"zip"}}

Proposed resolution

Lowercasing the package name

CommentFileSizeAuthor
#3 3256424-3.patch592 byteselber

Issue fork countup-3256424

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

louis-cuny created an issue. See original summary.

elber’s picture

Assigned: Unassigned » elber
elber’s picture

Assigned: elber » Unassigned
Status: Needs work » Needs review
StatusFileSize
new592 bytes
anagomes’s picture

Status: Needs review » Reviewed & tested by the community

Patch in #3 applies and fixes the uppercase characters.

louis-cuny’s picture

The patch #3 is not correct, check the MR instead

elber’s picture

Hello @louis-cuny what is "MR"? I didn't understand what is your desire. Can you help me, please?

andregp’s picture

Hi @elber I think you should have changed countUp.js to count-up.js not countup.js (As shown on issue description). The module description uses countup.js so it's probably preferable.
Also, as far as I know, MR means "merge request" so @louis-cuny is probably asking you to send a MR instead of a patch.

elber’s picture

Assigned: Unassigned » elber
Status: Reviewed & tested by the community » Needs work

b-prod’s picture

Status: Needs work » Fixed
elber’s picture

Assigned: elber » Unassigned

Status: Fixed » Closed (fixed)

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