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

ncm-metaconfig: fix a typo in 'actions' resource description #1663

Merged
merged 1 commit into from Mar 1, 2024

Conversation

jouvin
Copy link
Contributor

@jouvin jouvin commented Feb 29, 2024

Fix a small but misleading typo in ncm-metaconfig schema

@jouvin jouvin requested review from jrha and stdweird March 1, 2024 09:28
wpoely86
wpoely86 previously approved these changes Mar 1, 2024
@wpoely86
Copy link
Member

wpoely86 commented Mar 1, 2024

2024-03-01T09:46:25.3853668Z                 error(format('metaconfig element can only have one boolean conversion enabled, got %s', SELF));
2024-03-01T09:46:25.3855087Z                       ^^^^^^
2024-03-01T09:46:25.3855509Z 
2024-03-01T09:46:25.3856607Z ncm-metaconfig/src/main/pan/components/metaconfig/schema.pan:54: Redundant use of format within error or debug call
2024-03-01T09:46:25.3860275Z                 error(format('metaconfig element can only have one string conversion enabled, got %s', SELF));
2024-03-01T09:46:25.3861545Z                       ^^^^^^
2024-03-01T09:46:25.3861906Z 
2024-03-01T09:46:25.3862967Z ncm-metaconfig/src/main/pan/components/metaconfig/schema.pan:66: Redundant use of format within error or debug call
2024-03-01T09:46:25.3864273Z                 error(format('metaconfig element can only have one list conversion enabled, got %s', SELF));
2024-03-01T09:46:25.3865046Z                       ^^^^^^
2024-03-01T09:46:25.3865272Z 
2024-03-01T09:46:25.3865945Z ncm-metaconfig/src/main/pan/components/metaconfig/schema.pan:78: Redundant use of format within error or debug call
2024-03-01T09:46:25.3867071Z                 error(format('metaconfig element can only have one key case conversion enabled, got %s', SELF));
2024-03-01T09:46:25.3867819Z                       ^^^^^^

@jouvin Can you also fix those and make panlint happy?

@jouvin
Copy link
Contributor Author

jouvin commented Mar 1, 2024

Sure, will do it asap!

@wpoely86 wpoely86 merged commit 00c9881 into quattor:master Mar 1, 2024
2 checks passed
@jrha jrha added this to the 24.3 milestone Mar 8, 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.

None yet

3 participants