-
Notifications
You must be signed in to change notification settings - Fork 72
Chantier Permission Sets #1434
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
base: main
Are you sure you want to change the base?
Chantier Permission Sets #1434
Conversation
❌MegaLinter analysis: Error
Detailed Issues❌ SPELL / cspell - 29 errors
🤖 AI-Powered Fix Suggestions for SPELL_CSPELL (by openai gpt-4.1-mini)
// Before
Required: field.nillable ? 'No' : 'Yes',
// After
Required: field.nullable ? 'No' : 'Yes',
{
"words": ["sobj", "sofactoapp", "Profil_a_Associe", "Opportunité", "Produit", "Abonnement", "Accès", "Bénéficiaire", "tarification", "Profil", "Associe", "Personna", "Actif"]
} This approach keeps spell checking effective while respecting your codebase’s terminology. ❌ COPYPASTE / jscpd - 3 errors
🤖 AI-Powered Fix Suggestions for COPYPASTE_JSCPD (by openai gpt-4.1-mini)
Example refactor: // Before: duplicated code blocks
function processA() {
// duplicated logic here
}
function processB() {
// same logic repeated
}
// After: extracted reusable function
function commonProcess() {
// shared logic
}
function processA() {
commonProcess();
}
function processB() {
commonProcess();
} This reduces maintenance burden and improves code quality.
|
…upcom/sfdx-hardis into chantierPermissionSets
No description provided.