Skip to content

Remove .txt default files#1279

Merged
sjanzou merged 1 commit into
developfrom
default-txt-files
Nov 18, 2022
Merged

Remove .txt default files#1279
sjanzou merged 1 commit into
developfrom
default-txt-files

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Default .txt files were possibly unintentionally reintroduced by a merge removed. All default files should be .json files.

Possibly unintentionally reintroduced by a merge.
@cpaulgilman cpaulgilman added this to the SAM Fall 2022 Release milestone Nov 18, 2022
@cpaulgilman cpaulgilman requested a review from sjanzou November 18, 2022 00:44
@cpaulgilman cpaulgilman self-assigned this Nov 18, 2022

@sjanzou sjanzou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@brtietz, We need to have everyone do a clean checkout of SAM so we do not get any .txt defaults checked in with pull requests in the future.

@sjanzou sjanzou merged commit cd9ed2c into develop Nov 18, 2022
@sjanzou sjanzou deleted the default-txt-files branch November 18, 2022 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants