Skip to content

Storage buckets#41

Open
NilayYadav wants to merge 3 commits into
TrainLoop:mainfrom
NilayYadav:main
Open

Storage buckets#41
NilayYadav wants to merge 3 commits into
TrainLoop:mainfrom
NilayYadav:main

Conversation

@NilayYadav

Copy link
Copy Markdown
Contributor

Storage buckets

@@ -1,4 +1,4 @@
package trainloop
package main

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The package declaration has been changed from package trainloop to package main. This change will break any existing code that imports this module as a library using the trainloop package name.

If this file is intended to be used as a library (which seems likely given its location in an SDK directory), it should remain as package trainloop. The package main declaration should only be used if this file is intended to be the entry point of a standalone executable.

Consider reverting this change to maintain backward compatibility with existing code that depends on this package.

Suggested change
package main
package trainloop

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant