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

Update auto-close-pr.yml #160

Closed
wants to merge 1 commit into from

Conversation

mst-rajatmishra
Copy link

changes i tried was,
Variable Naming: I renamed the step ID from check_employee to check_member. This makes it clearer that the purpose is to check if the user is a member of the organization, not just an employee.
Error Handling: I switched from using console.log to console.error for logging errors. This is a better practice because it helps highlight issues more effectively when reviewing logs.
Consistent Return Values: I ensured that the output from the membership check is always a string. This makes it easier to compare values later in the workflow, reducing potential confusion.

Copy link

This pull request is being automatically closed because we do not accept external contributions to this repository.

@github-actions github-actions bot closed this Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant