Problem/Motivation

I see a PHPCS issue to fix in the pipeline execution: https://git.drupalcode.org/project/google_calendar_service/-/pipelines/4...

Steps to reproduce

N/A

Proposed resolution

Fix the issue mentioned above

Remaining tasks

N/A

User interface changes

N/A

API changes

N/A

Data model changes

N/A

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

danrod created an issue. See original summary.

danrod’s picture

Issue tags: +Novice

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

danrod’s picture

@julio_retkwa are you going to take this one?

julio_retkwa’s picture

Hi @danrod sorry, yes

I was not aware that you was working on this, sorry for jumping in

julio_retkwa’s picture

I created a MR against branch 8.x-2.x-dev but its failing on composer step
https://git.drupalcode.org/issue/google_calendar_service-3528488/-/jobs/...

I'm missing something here?

PS: I see there are some PHPCS issues on branch 1.x should those be addressed as well? Let me know
https://git.drupalcode.org/issue/google_calendar_service-3528488/-/jobs/...

Thanks!

julio_retkwa’s picture

Status: Active » Needs review
danrod’s picture

@julio_retkwa it is ok, work on the branch 8.x-2.x, which is the one I still want to maintain, if possible.

Thanks a lot for your work.

danrod’s picture

Status: Needs review » Needs work
danrod’s picture

@julio_retkwa I'm seeing the same composer issues when trying to create a MR against the 8.x-2.x branch, if you have some time to look on it, that would be great, if not, no worries, I'll look on it later.

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

ankitv18 changed the visibility of the branch 8.x-1.x to hidden.

ankitv18 changed the visibility of the branch 3528488-fix-phpcs-issue to hidden.

ankitv18 changed the visibility of the branch 3528488-fix-the-phpcs-8.x-2.x to hidden.

ankitv18’s picture

Status: Needs work » Needs review

MR!29 is against 2.x branch ~~ phpcs pipeline + composer changes are fixed now, hence moving into review.

danrod’s picture

Thanks a lot @nkitv18 and the others !

There's some CSPELL and other issues that I will fix now.

danrod’s picture

Status: Needs review » Needs work
danrod’s picture

Finally it's green now !!!

I'll move it to RTBC+1 and then merged to the corresponding branch.

Thanks a lot for everyone involved.

danrod’s picture

Status: Needs work » Reviewed & tested by the community
danrod’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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