Skip to content

Conversation

antonio-soares-altium
Copy link
Contributor

Added the import site package to add the script files through the import statement.

Cleared out login credentials.

Copy link
Contributor

@goncalo-dias-altium goncalo-dias-altium left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ricardo-dematos ricardo-dematos left a comment

Choose a reason for hiding this comment

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

LGTM

Removed authentication credentials and deployment sub-domain from some of the scripts as well as added header information.

These changes are only on the Children Suspect Warning script
Updated all scripts removing credentials and adding the import capacity for other script files
@ricardo-dematos ricardo-dematos dismissed stale reviews from goncalo-dias-altium and themself via b4c4646 February 10, 2023 00:53
@ricardo-dematos ricardo-dematos force-pushed the VALI-5456-Properly-Document-Existing-Scripting-Module-Scripts branch from 22d8473 to b4c4646 Compare February 10, 2023 00:53
@ricardo-dematos ricardo-dematos merged commit a034127 into develop Feb 10, 2023
@ricardo-dematos ricardo-dematos deleted the VALI-5456-Properly-Document-Existing-Scripting-Module-Scripts branch February 10, 2023 00:54
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.

3 participants