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

fix(review-bot): update review bot for new repo structure #4322

Merged
merged 5 commits into from
Jan 30, 2025
Merged

Conversation

RoRoJ
Copy link
Contributor

@RoRoJ RoRoJ commented Jan 30, 2025

Update review bot for new repo structure.

  • Addition of a new import statement (json) used in new method.

  • Addition of FILEPATH constant, since this is now used in two separate methods.

  • Addition and integration of a new method get_prod_cat_ref to create a dictionary where products are mapped to categories, as this can no longer be inferred from the filepath alone.

@RoRoJ RoRoJ added do not merge PR that shouldn't be merged before a specific date (eg release) status: tech review Waiting for technical review labels Jan 30, 2025
@RoRoJ RoRoJ requested a review from Slashgear January 30, 2025 10:20
@RoRoJ RoRoJ added type: maintenance Improvements or additions to documentation status: doc review Waiting for doc team review and removed do not merge PR that shouldn't be merged before a specific date (eg release) status: tech review Waiting for technical review labels Jan 30, 2025
@RoRoJ RoRoJ merged commit c2378ea into main Jan 30, 2025
6 of 7 checks passed
@RoRoJ RoRoJ deleted the MTA-5440 branch January 30, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: doc review Waiting for doc team review type: maintenance Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants