Skip to content

Conversation

antonag32
Copy link
Contributor

Closes #27.

This checks generates a message whenever a tag with 'oe_structure' in its classes has no id attribute.

@antonag32
Copy link
Contributor Author

Can you review @moylop260 @luisg123v

@antonag32 antonag32 force-pushed the oe-structure-anton branch 2 times, most recently from 98750ae to 8f8cf88 Compare June 23, 2023 19:39
@antonag32
Copy link
Contributor Author

Thanks for the review @luisg123v, I've made the corrections

@moylop260
Copy link
Collaborator

@luisg123v

Could you re-check it, please?

@luisg123v luisg123v self-requested a review June 27, 2023 01:59
@antonag32 antonag32 force-pushed the oe-structure-anton branch from 8f8cf88 to bb4623d Compare June 27, 2023 15:59
Closes OCA#27.

This checks generates a message whenever a tag with 'oe_structure'
in its classes has no id attribute.
Copy link

@luisg123v luisg123v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Just be aware you will need to consider this case in the xpath performance change (#79) if this one is merged before that one; or the other way around if that one is merged before this one.

@luisg123v
Copy link

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Sorry @luisg123v you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@luisg123v
Copy link

Don't blame me for trying 😄

@moylop260
Copy link
Collaborator

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch main-ocabot-merge-pr-78-by-moylop260-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jun 27, 2023
Signed-off-by moylop260
@moylop260 moylop260 merged commit e8998dc into OCA:main Jun 27, 2023
@moylop260 moylop260 deleted the oe-structure-anton branch June 27, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New linter: oe_structure element without specific id
4 participants