Comments (5)
I'm not sure what you mean but you can open a PR if you want.
you just can't commit the changes yourself.
what is the issue you are having?
from nws_alerts.
from nws_alerts.
@mdecourcy You're trying to push directly to someone else's repo you do not have access to. You have to fork, clone your fork, branch, commit, push, then PR.
The flow I use to committing and PRing to someone else's project is the following:
- Through GitHub web, you fork the repo. This makes a personal copy for yourself.
- You clone your fork.
- Create a new branch from
main
/master
or some other branch of interest. - Make your commits
- Push
- Visit your repo and you should get a yellow banner indicating you can PR your changes.
This is my flow, but I know there's quicker ways via GitHub's CLI and different ways to begin a PR. Explore around.
You will occasionally have to update your fork and there's a way to do it on the web, but typically I set up another remote on my local repo—like upstream
—, switch to your fork's main
, git fetch upstream
, git merge upstream/main
or something of that effect then git push
. Ideally it should fast-forward, but if it does something else or fails then the owner did a force push that broke the history or you have local commits that are conflicting.
from nws_alerts.
Ah I see, my mistake! First attempt at open source contributions. I'm used to just feature branching and PRing at work. I'll fork and make a PR now.
from nws_alerts.
@mdecourcy No problem, and that would be a valid way to do a PR as well on repos you have access to as well.
from nws_alerts.
Related Issues (20)
- UndefinedError: 'None' has no attribute 'split' HOT 2
- Inconsistent Entry Separator for event id's HOT 1
- Unavailable HOT 7
- Task exception was never retrieved error since update to version 2.6 HOT 5
- New error on 2023.2 on the integration which states it will fail in 2023.3 HOT 1
- Enhancement Idea: Add alert type filtering HOT 3
- Enhancement: Alert targeting using FIPS zone alone results in false alarm potential HOT 1
- Feature Request HOT 4
- Not showing up in integrations ui HOT 2
- Possible Variable issue - just FYI HOT 5
- Won't trigger automations HOT 3
- Package nws_alerts setup failed. Component variable Integration 'variable' not found. HOT 4
- Request to expand spoken attributes for sensor HOT 3
- Multiple alerts concatenated into long strings HOT 2
- Please consider adding 'onset' as an attribute HOT 1
- How to configure poll rate and timeout in manual config HOT 1
- API Errors Cause Integration State To Bounce Frequently HOT 2
- Attribute Value Data? HOT 6
- Documentation information HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nws_alerts.