Skip to content

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
Laure-di pushed a commit to Laure-di/docs-content that referenced this pull request Mar 11, 2025
)

* fix(review-bot): update for new structure

* fix(review-bot): review bot for new structure done

* fix(review-bot): test

* fix(bot): test

* fix(review-bot): final fix for indexes
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.

4 participants