Giter Club home page Giter Club logo

Comments (28)

popey avatar popey commented on May 12, 2024 4

A wild @popey appears!
I would agree that you might want to steer away from trademarked terms, if possible :)
If you're confident that you're safe there - or indeed have permission from the trademark holder, then by all means push the snap to the store.

from teams-for-linux.

popey avatar popey commented on May 12, 2024 2

@benyanke there are a few other snaps which are essentially web wrappers in the store. That's cool beans to me.

from teams-for-linux.

popey avatar popey commented on May 12, 2024 2

🤷
I am neither a lawyer, nor do I play one on TV.

from teams-for-linux.

julian-alarcon avatar julian-alarcon commented on May 12, 2024 2

Ben, what is your Snapcraft registered email to add you to the admins? Please don't release on stable/beta channel, keep it for now on edge

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024 2

@benyanke and @julian-alarcon , great work guys.

happy to be added as a collaborator to the snap.

I normally create a branch with the next version number, make my changes, and PR to master. In this case is 0.1.17, as this still to be released.

Travis gets a bit temperamental if I update the version directly in master, but sometimes I do if I get a direct pull requests.

Happy to accommodate another structure and to move from travis to gitlab if it eases the process (travis can be a bit of a pain)

Over the years I have learn to keep it simple, so maybe the best approach is to only have stable and edge. At least until we see the need of other channels.

But I am don't know much about snap channels and how people use them.

Regarding what to publish to what, I will say that, if we follow the previous branching strategy, we should be able to configure the CI to publish to edge (for branches) and stable (for master) by checking the branch the build is happening.

Again, lets keep it simple, so that might be version 2 or 3. Get it release to edge and we can stabilise later.

Happy to discuss further and to accommodate to other options. Thanks again!!

ps: trying to help but pretty busy currently with work and life!

from teams-for-linux.

julian-alarcon avatar julian-alarcon commented on May 12, 2024 1

Right now I'm testing snaps builds, but yeah, the idea will be to publish it in the Snap Store, I just don't know if there are any issues if in the name of the snap/app we refer to an specific defined brand (Microsoft Teams / Teams For Linux).

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024 1

Perhaps https://twitter.com/popey might be a good person to kick this discussion off with - no doubt it would be a great add to the store, but getting in trouble and getting it removed isn't good either.

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024 1

I added you and Julian as collaborators. Let me know if that does it. 1st 2 collaborators! I might have a bit of time this Saturday to play around with it.

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024

Happy to support you on that. I can't test the snap packages myself so I can't really take the task. Anyone interested?

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

If you wire up the new-snap-on-commit CI piece, so the snap stays up to date (one of the big reasons for doing it), I can test it.

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

@popey might tag him too

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

Thanks @popey - given those constants, is there any way to publish a tool like this (if you haven't looked closer, it's basically a native-ied web wrapper for MS teams) perhaps by naming it something else? Or is a tool like this simply off limits based on it's content?

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024

maybe teams-for-linux will be better... anyone from MS watching can let us know otherwise. ;)

@popey Thanks for the input.
@benyanke and @julian-alarcon , I leave it in your capable hands, while I try to fix/improve other stuff. If needed to move from Travis to Gitlab let me know. I don't have any strong attachment with travis.

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

I think any CI system should work, but I'll dive in and see to be sure. We may have to coordinate to get it set up but I'm sure it will work fine. If it's easier, I could also fork the repo, connect that one to the store, and add you as an admin on the repo, so you don't need to worry about the CI and connection to the snap store. Whichever works for you.

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

So as long as we call it something like "Teams for Linux" instead of "Microsoft Teams", we're good?

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

Ha - fair enough. Thanks again, @popey.

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

I've registered the snap name teams-unofficial and will begin attempting to get this set up.

from teams-for-linux.

julian-alarcon avatar julian-alarcon commented on May 12, 2024

I registered the teams-for-linux. @popey / @IsmaelMartinez which one do you think that is better to use, teams-unofficial or teams-for-linux?

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024

For consistency sake... teams-for-linux

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024

But happy with either

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

Sounds good - do you want to add me as a co-admin @julian-alarcon on the snap side, or do you plan to run it solo? I'm fine with either.

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

Also, related:

https://docs.travis-ci.com/user/deployment/snaps/

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

containerized snap builds tested and working via ./dockerBuildSnap.sh, allowing easy reproducible builds

from teams-for-linux.

benyanke avatar benyanke commented on May 12, 2024

Emailed you directly - sounds good about the channels.

from teams-for-linux.

julian-alarcon avatar julian-alarcon commented on May 12, 2024

Ok, I just uploaded the version 0.1.17 to make some testing (edge channel), I needed to make a little change to the executable (I will upload it to an specific branch and ask for merge on master(?) @IsmaelMartinez do you want to create a develop branch or a new 0.1.18 branch?)

snapcraft push --release=edge teams-for-linux_0.1.17_amd64.snap
Preparing to push 'teams-for-linux_0.1.17_amd64.snap'.
After pushing, an attempt will be made to release to 'edge'
Pushing 'teams-for-linux_0.1.17_amd64.snap' [================================================================================================================================================================] 100%
Processing...|                                                                                                                                                                                                     
Ready to release!
Revision 1 of 'teams-for-linux' created.
Track    Arch    Channel    Version    Revision
latest   amd64   stable     -          -
                 candidate  -          -
                 beta       -          -
                 edge       0.1.17     1
The 'edge' channel is now open.

Installation and execution worked fine (adding, for now, the edge option):

sudo snap install teams-for-linux
error: snap "teams-for-linux" is not available on stable but is available to install on the
       following channels:
       edge       snap install --edge teams-for-linux
       Please be mindful pre-release channels may include features not completely tested or
       implemented. Get more information with 'snap info teams-for-linux'.
1 julian@mypc:~/Code/teams-for-linux/dist$ sudo snap install teams-for-linux --edge
teams-for-linux (edge) 0.1.17 from Julian Alarcon (alarconj) installed

Once we have the CI/CD in place we can then deliver to other channels. Also @IsmaelMartinez , do you want to use all the channels (stable/candidate/beta/edge) or any specific for this?

Also, @benyanke I added you as collaborator for this snap, if you want to keep making test, remember to only upload to edge for now.

from teams-for-linux.

julian-alarcon avatar julian-alarcon commented on May 12, 2024

Of course @IsmaelMartinez that I can add you as a collaborator! (I can even ask to pass you the authoring if you want to).

Just give me your mail on Snapcraft https://snapcraft.io/account (You can mail me, my address is in my profile)

About the number of channels, 2 is ok (edge/stable) from my side. So, to keep things a bit cleaner, after a release, can you create the next branch version to push features/bugfixes to that specific branch and keep master clean?

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024

yep, I tend to do that. 0.1.17 is, at this moment in time, a bit out of date with master. I will just do a quick refresh to it. No need to add me as author, this is/should be a team effort. ;)

from teams-for-linux.

IsmaelMartinez avatar IsmaelMartinez commented on May 12, 2024

I have updated the 0.1.17 branch with the latest code.

I am preparing a release so I am going to close this issue and create another regarding the automation of the snap release (integrate with Travis). Also, we will stop spamming @popey.

I will also create a wee project in here so we can track more easily what goes in version 0.1.18. Might be better not to commit to master directly but do it in the 0.1.18 (I will create a branch as soon as the 0.1.17 is out of the door).

Thanks,

from teams-for-linux.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.