This repository was archived by the owner on Sep 1, 2022. It is now read-only.
Promote changes from Dev to Master #32
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
In the Export() sub, an unhandled error would occur if the "References" or "Module Paths" properties were not set in the JSON configuration file. The problem was handled by making these properties optional. If they are not present, the Export sub will not perform the action related to each property. This means the behaviour for an empty property is identical to no property. This is the behaviour already implemented in the Import sub. I forgot to change the Export sub when I did the Import implementation.
Add instruction to the README to enable the VBA project model trust.
When an error occurs an error dialog will show as before. However this dialog will ask if the user would like to enter the debugger. The error handler is still only implemented in the same procedures as before. Before putting it in all procedures I want to wait for another PR to get merged so that I can rename the exitSub label to exitProc.
This is a fix for an issue mainly with sheet modules. It is normal for sheet modules not to have any code. In this case it is distracting to have empty files in the project directory just for these. Instead they shouldn't be added to the configuration file. I hope this won't cause any problems with modules left intentionally empty.
enable VBA project model trust instruction
Allow debugging from error dialog
Only add non-empty modules to configuration
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.