-
Notifications
You must be signed in to change notification settings - Fork 25
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
Update build files #73
Conversation
a6a73ad
to
bd70324
Compare
17b79f6
to
dfbc510
Compare
dfbc510
to
a330532
Compare
@@ -229,200 +229,6 @@ | |||
See the License for the specific language governing permissions and | |||
limitations under the License. | |||
|
|||
Includes the INI library (https://github.com/go-ini/ini) distributed under the following license: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to add the newly added libs https://github.com/nginxinc/nginx-asg-sync/blob/master/go.sum ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure how this works...like do we need to include Apache 2.0 multiple times if used by multiple libs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, I think if we just concatenate all licenses, that would be the easiest way. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated it
Co-authored-by: Michael Pleshakov <pleshakov@users.noreply.github.com>
Update build files before release