Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent "usermod: no changes" notice #152

Closed
wants to merge 1 commit into from
Closed

Conversation

janw-me
Copy link

@janw-me janw-me commented Jun 14, 2022

I see this notice very often, including in issues as part of debugging.
It also triggered on all of my dynamic tooling.
So rather than just silencing the error I wrapped it in an if to see if it's even needed in the first place.

I saw it every dynamic event. rough example for lando composer install

events:
  pre-composer:
    - appserver: echo DO THINGS

I see this notice very often, including in issues as part of debugging.
It also triggered on all of my dynamic tooling.
So rather than just silencing the error I wrapped it in an `if` to see if it's even needed in the first place.
@netlify
Copy link

netlify bot commented Jun 14, 2022

👷 Deploy request for lando-cli pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 475149f

@pirog
Copy link
Member

pirog commented Sep 22, 2023

@janw-me finally at a point in our repo reorg where this is possible although it lives in a different repo now. Going to close this but ive applied your commit over here lando/core#51

@pirog pirog closed this Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants