Problem/Motivation

Add tugboat support for easier manual review and testing in issues

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork list_formatter-3581473

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

anybody created an issue. See original summary.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Title: Add tugboat support for easier manual review and testing in issues » Add composer.json and tugboat support for easier manual review and testing in issues
Issue tags: +Novice

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

sapnil_biswas’s picture

sapnil_biswas commented
Assigned: Unassigned » sapnil_biswas

sapnil_biswas opened merge request !18

sapnil_biswas’s picture

sapnil_biswas commented
Assigned: sapnil_biswas » Unassigned
Status: Active » Needs review

@Anybody marking this for review please let me know if there any further necessary changes required

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

@sapnil_biswas thanks, I corrected the implementations, please look into https://git.drupalcode.org/project/list_formatter/-/merge_requests/18/di... to learn for future implementations.

Template from drupal.org documentation should be used, please.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Needs work

@sapnil_biswas would you be interested to check and fix code style here now, looking at the warnings in the CI pipeline?

sapnil_biswas’s picture

sapnil_biswas commented
Status: Needs work » Needs review

@anybody
Thank you sir for your feedback and advice it will surely help me in my future contributions,
I tried fixing the warnings of the pipeline now only the phpstan is faling but it doesn't seem like it is failing due any changes in this scope of the issue please let me know if there is anything I am missing

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Reviewed & tested by the community

Thanks @sapnil_biswas!

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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