Support for global RPGLINT configuration path

[francois-rpg]

Changes

This PR adds support for specifying a global RPGLINT config path using the new setting rpgle.globalLintConfigPath.
If checked, linter configs for libraries will be based on IFS's /etc/vscode/rpglint.json.
This helps enforce company-wide lint standards.
Exclude libraries also added, to enable user to add libraries they don't want linted, e.g. browsing prod libraries' source.

Checklist

• [Y] have tested my change
• updated relevant documentation
• Remove any/all console.logs I added
• eslint is not complaining
• have added myself to the contributors' list in the README
• for feature PRs: PR only includes one feature enhancement.

This is my first pull request to anything on github, ever. Please let me know if and where I went wrong so I contribute cleanly in the future.

[worksofliam]

Hey, this is a great start! I am going to need some files removed from this PR:

• package-extension.yml - please don't add any new actions to this PR. Perhaps in a separate one if this is something you want to see.
• Please remove all .vsix files. We don't want to check them into the repo.

Thanks! Then I will go in for a second review 😄

[francois-rpg]

Requested changes completed.
Thank you for the feedback.

[worksofliam]

Sorry this has token so long to get to.

1. Can you please fix the formatting in your new changes? They don't match the indentation of the rest of the file.
2. Can you fix the conflicts

Thanks!

[SanjulaGanepola]

I think we should standardize with Code for IBM i and make use of the /home/<user>/.vscode directory to store this configuration file much like the settings.json and profiles.json. We can also use the ConfigFile APIs. What do you think?

[francois-rpg]

I think we should standardize with Code for IBM i and make use of the /home/<user>/.vscode directory to store this configuration file much like the settings.json and profiles.json. We can also use the ConfigFile APIs. What do you think?

That is a good idea. I've been thinking about this change, should there also be a switch on/off?
If you're browsing your main repo with old programs you don't necessarily want linting.
Thoughts?

[SanjulaGanepola]

Also I made a typo in my original comment. I meant to type /etc/vscode instead of /home/<user>/.vscode as this is a shared global config directory in Code4i. As long as you use the ConfigFile API, it will automatically use this directory.

should there also be a switch on/off?

Great point actually. I'm wondering what would be the best way to support this though. What where you thinking?

[francois-rpg]

I have made final changes on my side, making it easier to configure the global files (Not uploaded yet, need testing first).
There is an issue with Mapepire at the moment.
I logged the below and will await a fix before finishing off this PR.
#532

[francois-rpg]

Also I made a typo in my original comment. I meant to type /etc/vscode instead of /home/<user>/.vscode as this is a shared global config directory in Code4i. As long as you use the ConfigFile API, it will automatically use this directory.

should there also be a switch on/off?

Great point actually. I'm wondering what would be the best way to support this though. What where you thinking?

Hi Sanjula,
I hope you're well.

Do you think switching to this method would solve for the existing Mapepire issue? Or should I rather wait for them to fix the one I logged?

[francois-rpg]

@worksofliam @SanjulaGanepola
Changed config path to IFS.
Tested successfully.