Problem/Motivation

A space at the end of a YMAL file is causing errors in our DevOps since its windows based and does not allow filename to end in a space.
error: invalid path 'docroot/modules/contrib/flood_control/migrations/state/flood_control.migrate_drupal.yml ' (note space and end of filename)

SEE https://git.drupalcode.org/project/flood_control/-/blob/2.3.x/migrations... how the file name ends in a space.

Steps to reproduce

Proposed resolution

Remove ending space from filename

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork flood_control-3337620

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:

About issue forks

Comments

NicholasS created an issue. See original summary.

NicholasS opened merge request !35

nicholass’s picture

nicholass
Primary language English
Location FL
commented
Status: Active » Needs review
nicholass’s picture

nicholass
Primary language English
Location FL
commented
Related issues: +#3337866: Will not install under drupal 10, +#3337292: Composer fails to update module with error
nicholass’s picture

nicholass
Primary language English
Location FL
commented

For Others who are not used to the new merge request process here is how you composer patch this. However on windows it still may not patch correctly so you may have to manually remove the space off the yaml. Hopefully this gets fixed soon.

"drupal/flood_control": {
                "YAML filename ends in space, git checkout in windows fails. https://www.drupal.org/project/flood_control/issues/3337620": "https://git.drupalcode.org/project/flood_control/-/merge_requests/35.diff"
            },
f1mishutka’s picture

f1mishutka commented
Status: Needs review » Reviewed & tested by the community

fabianderijk made their first commit to this issue’s fork.

fabianderijk’s picture

fabianderijk
him/his
Primary language Dutch
Location Alphen aan den Rijn
commented
Status: Reviewed & tested by the community » Fixed

Thanks, I have merged this just now

matus grinvalsky’s picture

matus grinvalsky commented

Thanks! Works for me as well.

Status: Fixed » Closed (fixed)

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