github / safe-settings

ISC License
563 stars 138 forks source link

add outer try-catch fix issue #545 #546

Closed decyjphr closed 6 months ago

decyjphr commented 8 months ago
decyjphr commented 6 months ago

Oh! Sorry. I must’ve accidentally removed it. Can you create a pR and I’ll merge it. I am traveling and using my phone so it would be easier that way.

On Fri, Dec 8, 2023 at 7:11 AM Sergio Valverde @.***> wrote:

@.**** commented on this pull request.

In lib/settings.js https://github.com/github/safe-settings/pull/546#discussion_r1419843270:

  • rulesets: require('./plugins/rulesets'),
  • environments: require('./plugins/environments')
  • rulesets: require('./plugins/rulesets')

@decyjphr https://github.com/decyjphr did you remove the environments plugin ?

— Reply to this email directly, view it on GitHub https://github.com/github/safe-settings/pull/546#discussion_r1419843270, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANXBBBTPECPPGRGU7UQ6USTYIJV4DAVCNFSM6AAAAAA6YIHNBSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONZRGQYTMNBVGM . You are receiving this because you were mentioned.Message ID: @.***>

-- Yadhav Jayaraman, Github Expert Services