text:
🛑 invidious.slipfox.xyz is down

In 58fac27, invidious.slipfox.xyz (https://invidious.slipfox.xyz/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: invidious.slipfox.xyz is back up in b3ea6b7.


text:
🛑 search.mascotboi.xyz is down

In 3f92a75, search.mascotboi.xyz (https://search.mascotboi.xyz/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: search.mascotboi.xyz is back up in 4f8eec0 after 35 minutes.

source: gharchive/issue

repo: gitetsu/librex-instances-upptime

license: MIT

license_type: permissive

license_source: github-api


text:
Add support for Elixir Ecosystem for dependency graphs

You already support Erlang (registry: https://hex.pm/) to build dependency graphs so you should implement the same functionality for Elixir projects which use the mix.exs script in root of a project (and within nested application root folders) and they mostly use the same registry; hex.pm (although they can link to github or or other repos directly)

Would love this, please consider adding Elixir to the supported ecosystems

Would love this too, please.

Is this open source?
Can we do something to implement the dependency graph in elixir?

👋  Hi from the dependency graph team -- Elixir support is near the top of our list for additional ecosystems to support, but we don't currently have a timeline.
Our core isn't open source, but we do have the dependency submission API that can submit dependencies for other ecosystems. Some folks have created GitHub Actions to do this for various ecosystems -- if anyone wants to go that route, we have the dependency submission toolkit that can help with developing actions for dependency submission.

Good to know you're working on it. It'll truly help having it.

Interesting, It'll truly useful and help.

Really, if it supports Erlang, it should support Elixir, after all, the Elixir people wrote hex.pm.

I'd also like to support this feature. Actually, there are several languages which run on the Erlang VM. Ideally, the graphs would support most of them. And a pony... :)

Would love to see Elixir support too

@courtneycl

Our core isn't open source, but we do have the dependency submission API that can submit dependencies for other ecosystems.

Unfortunately, the submission API does not support Hex purls. All reported dependencies are show as pkg:unknown.
Hex purl Spec: https://github.com/hexpm/specifications/blob/main/package-url.md
Action / Tool: https://github.com/maennchen/mix-dependency-submission
Dependencies: https://github.com/maennchen/mix-dependency-submission/network/dependencies?page=2

When downloading the SBoM:
// ...
{
  "SPDXID": "SPDXRef-unknown-dialyxir-1.4.4",
  "name": "unknown:dialyxir",
  "versionInfo": "1.4.4",
  "downloadLocation": "NOASSERTION",
  "filesAnalyzed": false,
  "supplier": "NOASSERTION",
  "externalRefs": [
    {
      "referenceCategory": "PACKAGE-MANAGER",
      "referenceLocator": "pkg:unknown/dialyxir@1.4.4",
      "referenceType": "purl"
    }
  ]
}
// ...

Expected:
// ...
{
  "SPDXID": "SPDXRef-hex-dialyxir-1.4.4",
  "name": "hex:dialyxir",
  "versionInfo": "1.4.4",
  "downloadLocation": "NOASSERTION",
  "filesAnalyzed": false,
  "supplier": "NOASSERTION",
  "externalRefs": [
    {
      "referenceCategory": "PACKAGE-MANAGER",
      "referenceLocator": "pkg:hex/dialyxir@1.4.4",
      "referenceType": "purl"
    }
  ]
}
// ...

Reported:
https://github.com/maennchen/mix-dependency-submission/actions/runs/11507378792/job/32033446490
{
  "version": 0,
  "metadata": {},
  "ref": "refs/heads/main",
  "manifests": {
    "mix.exs": {
      "name": "mix.exs",
      "file": {
        "source_location": "mix.exs"
      },
      "metadata": {},
      "resolved": {
         // ...
        "dialyxir": {
          "scope": "development",
          "metadata": {
            "name": "dialyxir"
          },
          "dependencies": [],
          "package_url": "pkg:hex/dialyxir@1.4.4",
          "relationship": "direct"
        },
        // ...
      }
    }
  },
  "detector": {
    "name": "mix_dependency_submission",
    "version": "1.0.0-beta.1",
    "url": "https://github.com/maennchen/mix-dependency-submission"
  },
  "sha": "71d59cd5aadc839808701997f2c9b6f50513eb35",
  "job": {
    "id": "escript",
    "correlator": "Main Branchescript"
  },
  "scanned": "2024-10-24T21:01:37.127673Z"
}

authors: InFo555 RichMorin SupaMic binarytemple cj1128 courtneycl dimitarvp dnovais maennchen pgbezerra yammine

repo: github/advisory-database


text:
CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled

Hello,
My CodeQL analysis suddenly stopped working in following run: https://github.com/swagger-api/apidom/actions/runs/4112641841/jobs/7098389969#step:4:8259
It fails specifically on:

and says following: CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled.
I'm using advanced setup and custom codeql.yaml workflow file. I've consulted https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/troubleshooting-the-codeql-workflow and determined that I don't have default setup enabled.
Here is my code scanning settings:

I'm really now sure at this point what is happening and how to get CodeQL working. I've tried to disable and re-enable the workflow, deleted the workflow file and re-created it again. Interesting about this is that CodeQL in PRs is working, but it just doesn't work on main branch.
Any advice please?

:wave: @char0n for the report! :heart: This has highlighted a bug that was recently introduced. We are working on a fix.

This should be fixed now, please let me know if it is not the case, otherwise, feel free to close the issue. Thank you!

Yep, I can confirm, everything now works as expected. Thank you!

@marcogario Looks like the bug is not fully resolved if you look at https://github.com/anmol098/waka-readme-stats/pull/387
@char0n do you mind re-opening this ticket?

This issue hasn't been resolved, see https://github.com/StadGent/node_service_adlib-backend/pull/97
https://github.com/StadGent/node_service_adlib-backend/actions/runs/4419164254/jobs/7747694733

@aravindvnair99 all right, I'm reopening, but as I mentioned in https://github.com/github/codeql-action/issues/1528#issuecomment-1422280604, it has been resolved for me.

Same issue here: https://github.com/StadGent/node_service_adlib-backend/pull/104

:wave:  @MPParsley  Thanks for the feedback.
Please note that it is intentional that you are not able to run both an analysis via an advanced configuration and default setup, you can only run one of the two. If you want to continue using default setup, please either delete the yml workflow file or disable it.
The original issue reported here was due to a bug that even when default setup was disabled the yml upload was being rejected.  I will change the title of the issue to make this more clear.
We are working on improving the flow for converting from advanced to default , so that hopefully things are smoother here.
I will close the issue, but feel free to re-open it and provide more information, if this does not clarify.

repo: github/codeql-action


text:
Temporarily include the Actions extractor with the Action

This PR embeds the new experimental actions extractor to the CodeQL Action repo itself, and updates the init action to add that directory to --search-path if the actions language is enabled.
This is a hack to make the actions extractor available without it actually being in the bundle. This is needed for some internal testing over the next couple of weeks, after which the extractor will be in the bundle and we can remove this hack.
The extractor sources embedded here are just the current version copied directly from github/codeql.
I've already verified that this works correctly on a test repo with Advanced Setup.

This looks reasonable. Can you share a link to the run in the test repo?

Successful run in semmle-code: https://github.com/github/semmle-code/actions/runs/11731165575/job/32680646659

source: gharchive/pull-request

authors: aeisenberg dbartol


text:
Support for secret scanning alerts

This PR makes the action also synchronize secret scanning alerts.
Note that the secret scanning api only allows fetching secret alerts from private repositories. For public repositories, we just assume an empty result list.
To achieve this, I had to refactor parts of ghlib.py and change the format of the state files (files with the old format will be automatically converted). I took some care to not disturb existing synchronization projects, but would be happy to hear about cases were this PR impacts existing projects negatively.
I'll give this a bit more testing, but wanted to put it out here for initial thoughts.
@cmboling

I got some time to buckle down on testing this Thursday and Friday! Just skimmed thru the changes and looks good so far. I'll update you here by end of Friday (PST).

Awesome, appreciate it! I am still working on the webhook parts for this, so it's probably best to wait with testing until this PR is complete, though. I'll update this soon. Cheers!

I'm planning to refactor this after releasing the breaking changes here: https://github.com/github/codescanning-jira-integration/issues/16

Resolved conflicts; need to do thorough testing of this:

[ ] We can sync secret scanning alerts and update code scanning alerts like usual
[ ] We can sync updated secret scanning alerts, as well as update code scanning alerts like usual
[ ] We can successfully add integration to a new repo and sync alerts and satisfy the above two criteria as well

Thanks @cmboling ! I'll see whether I can test this, but it would likely be next week.

@zbazztian tested this pretty thoroughly! LGTM. 👍🏼

@cmboling Looks good to me. Did we ever think about having versioned branches? I guess currently people are using the action off of the main branch, so perhaps we should at some point transition to having v1, v2, ... branches / tags, to make this more stable, especially when we introduce major features like this.
However, due to lack of time, I'll merge this for now. Thanks for bringing this back to life!

Hey @zbazztian! Just got back from OOO a few days ago! Thanks for merging. At the moment I've been moving the v1 branch up to main; I thought about making the v2 branch to have the secret scanning alerts, but that would mean users will have to submit another set of changes to take advantage of a new branch. If we make other significant changes from this point forward, let's make that the v2 branch. 👍🏼

authors: cmboling zbazztian

repo: github/codescanning-jira-integration


text:
Hello, I work on an open-source project and we want to consistently, and without minimal specialization, check for all links in a collection of Markdown documents for our community and our staff developers. The are only a few of the latter, and want to be able to keep for user contributions as much as our own changes. It appears from GitHub Actions hosted runners (we don't do self-managed yet) we often get HTTP 403 errors and rate limited. This appears to be separate of rate limiting for the API as I discussed in github.community, where they also recommend I ask about it here.

Hello, I work on an open-source project and we want to consistently, and without minimal specialization, check for all links in a collection of Markdown documents for our community and our staff developers. The are only a few of the latter, and want to be able to keep for user contributions as much as our own changes. It appears from GitHub Actions hosted runners (we don't do self-managed yet) we often get HTTP 403 errors and rate limited. This appears to be separate of rate limiting for the API as I discussed in github.community, where they also recommend I ask about it here.
This recently popped up again today.
https://github.com/usnistgov/OSCAL/runs/6290851865?check_suite_focus=true
So some questions and maybe I open up an improvement issue if you agree this is worthwhile.

Is this rate limiting from hosted GHA runners intentional?
Is the rate limiting able to lifted from GHA runners by passing an appropriate token? It appears initial test attempts suggest no.
If we ask you consider switching the rate limiting response error code in HTTP from 403 to 429, will that be something that is possible? That seems to be appropriate for this context and would allow us to use a popular link checker for Markdown with existing configuration with retryOn429 and aliveStatusCodes behavior that is more in line with the nature of the error response, and not a HTTP 403 response which is a little more blunt.

Again, we'd like to make this work with low effort for our team and not more specialized code for just docs.github.com, but not sure how to proceed. It broke builds enough we exempted all of docs.github.com, but that obviously doesn't help and kind of defeats the purpose if we change them or links for GH do change.
Sincerely,
Your Friends at NIST
Originally posted by @aj-stein-nist in https://github.com/github/docs/discussions/17620

Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

Hello @91CH, are you one such docs team member, this has been failing our link checks in our builds of our public docs before a release a little more frequently than normal as of late. Does the conversion to an issue mean 404->429 is even something on the table for discussion?

@aj-stein-nist The author who shared your discussion is most likely a spammer and I am so sorry about this confusion. I have let the team our engineering team know about the discussion post you opened 💖.
I will be closing this issue now since this is re-posted and considered spam. Thank you for your patience 💛 .

@aj-stein-nist The author who shared your discussion is most likely a spammer and I am so sorry about this confusion. I have let the team our engineering team know about the discussion post you opened 💖.
I will be closing this issue now since this is re-posted and considered spam. Thank you for your patience 💛 .

All good, no worries! I will continue to follow the discussion posts instead. 👋

authors: 91CH aj-stein-nist ramyaparimi

repo: github/docs


text:
Add warning about ssh hardware security keys.

Why:
Closes [issue link]
I've tried signing commits with an hardware generated ssh ed25519 key and I get an error message. Maybe this is planned to be supported, but as of git 2.38, it doesn't seem to work. See https://docs.github.com/en/authentication/connecting-to-github-with-ssh/generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent#generating-a-new-ssh-key-for-a-hardware-security-key for what I'm talking about.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Just adding a sentence.
Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

@nicolas-graves Thanks so much for opening a PR! I'll get this triaged for review :zap:

Sorry, that was a configuration error (the first tutorial doc I used didn't gave info about the key:: prefix which is indeed now in docs).

@nicolas-graves Thanks for coming back and updating us! We appreciate that. And thank you again for your passion and interest in improving the docs!


text:
Added the "cache" folder entry from "App_data"

The cache folder is generated and don't need to be commited

@mrfloo any pointer to Umbraco docs about these rules, so we can point users to them to understand more about the setup?

Hi @shiftkey,
I could'nt find a nice documentation on this. This folder is caching all the content since version 7.1.1
As explained in this article about Azure but that's happen outside of azure as well: http://www.orbitalgamestudios.com/blog/umbraco-cache-size-azure/

Closing in favour of #1661 which has this rule (and others) and is currently under review.

authors: mrfloo shiftkey

repo: github/gitignore

license_source: bigquery


text:
feat: Add missing executables and generated

Reasons for making this change:
There are missing sections that should be added to .gitignore
Links to documentation supporting these rule changes:
https://gohugo.io/hugo-pipes/introduction/

@erdtsksn any links to StackOverflow discussions or other documentation about this change?

@erdtsksn any links to StackOverflow discussions or other documentation about this change?

https://discourse.gohugo.io/t/why-does-hugo-create-the-resources-directory/13021/4
https://discourse.gohugo.io/t/why-does-hugo-create-the-resources-directory/13021/3
bep who is one of the core contributor said you may add it to git if you like. But also he stated it is a resource cache. In my opinion it should be in gitignore if it is cache.
https://github.com/netlify-templates/one-click-hugo-cms/tree/master/bin
Netlify have a one-click template to create hugo websites. As you may see in the link above. They include hugo binaries inside bin folder.

@erdtsksn great, thanks for the extra context!

authors: erdtsksn shiftkey


text:
Option for hub browse: Print url without opening browser

I'd like to be able to share the url for the last commit quickly, but there doesn't appear to currently be a way to simply get the url that's opened by hub browse before it's opened in a browser, to be piped to a clipboard manager. Can this be implemented please?

This should do it: hub browse -u | pbcopy
Docs:
$ hub browse -h
usage: git browse [-u] [[<USER>/]<REPOSITORY>|--] [SUBPAGE]

[...] With "-u", outputs the URL rather than opening the browser.

Oh, I tried various commands looking for the help function. Thanks.

repo: github/hub

license: mit


text:
Add Dhall language support.

Fixes dhall-lang/dhall-lang#198.
There’s not yet a Dhall grammar in a supported format, so it uses Haskell for now.

Description

Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Adhall+NOT+nothack
here are Harvester results:

Unique repos: 489
Unique users: 314

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/dhall-lang/dhall-lang/blob/master/tests/parser/success/largeExpressionA.dhall
https://github.com/dhall-lang/dhall-lang/blob/master/tests/normalization/success/remoteSystemsA.dhall

[ ] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

I used Harvester to search for popularity, and after running all the processing there the result summary is:
Unique repos: 489
Unique users: 314

I cannot seem to get linguist set up on my machine so I'm not able to run the proximity test script, but here's some colour suggestions:

#dd1a11
#d1a11f
#dfafff

@sellout update: I did get the tests running, and so far #dfafff is the only color that passed. Would that be fine?

authors: f-f sellout

repo: github/linguist


text:
Expose information about development dependencies in Dependabot alerts

Summary
In ecosystems like npm where developers can indicate that a certain dependency is used during development only (a devDependency), developers have asked for a way to understand when Dependabot alerts are related only to a development dependency. In this initial ship, we expect to add support for a label on alerts that concern development dependencies to make it easier to filter, triage, or sort alerts.

This has shipped to dotcom! 🚢 https://github.blog/changelog/2022-06-23-dependabot-alerts-filter-alerts-by-the-scope-of-the-dependency-runtime-and-development/

🚢 This has shipped with the GHES 3.7 release: https://docs.github.com/en/enterprise-server@3.7/admin/release-notes

authors: ankneis github-product-roadmap

repo: github/roadmap


text:
Unable to login to github

Describe the bug
I'm unable to log-in to github through Vscode. I don't have error message. The login step in my browser is on success, but redirect to vs-code don'"t log me.
I have try the second way this bottom notification, but same effect.

To Reproduce
Steps to reproduce the behavior:

Install WSL
Login trough WSL

Expected behavior
Login to github and show all workflow
Screenshots

Extension Version
v0.25.3
Additional context
Add any other context about the problem here.

Here too.
Ubuntu 22.04.2 LTS

Same issue, MacBook Pro M1, Ventura 13.0, vscode 1.76.0. I get redirected to Github, proceed with login normally and when I confirm opening the vscode link, vscode still shows "Sign in to GitHub" with a notification saying that if I have trouble, I can try different method. Clicking yes opens http://127.0.0.1:53220/ that only says "You are signed in" and that's it, nothing changes.

I somehow fixed it - I opened another project/repo in different window and instead of a "Sign in to Github" button I got "Unable to connect to Github API. Do you have internet connection?". I then switched to the previous project, reloaded window and it loaded fine.

Same problem((
MacOS 13.3 (22E252)
Version: 1.77.1 (Universal)
Commit: b7886d7461186a5eac768481578c1d7ca80e2d21
Date: 2023-04-04T23:23:33.779Z (6 days ago)
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: 10.2.154.26-electron.0
OS: Darwin arm64 22.4.0
Sandboxed: No

@fox-john - I just tried this in WSL Ubuntu 22.04 and it worked for me, here is a demo of that specific environment:

We have also tested this on Windows, Mac Intel, and Mac M1 and it worked.
@fox-john @inomdzhon @OpenCode @MetRonnie -
Would you please try:

Uninstall v0.25.3 and install the new version v0.25.4 and login
Try reloading VS Code

Try disabling any other extensions to see if they are interfering. Then reload VS Code.
Do you have any Wiki repos in your GitHub repo? If so these could be the issue and we should make a bug for this.
Lastly, could you check the Output tab in VS Code and select GitHub Actions in the dropdown?  Could you send us a copy of any errors there?

Please let me know if any of these steps work, if so which one. And if not, what error messages you have so I can debug. Thank you so much for your help!

I am able to login with v0.25.4 👍

I'm using 0.25.4. When I try to sign in my browser is opened to https://github.com/login/oauth/authorize?client_id=xxx and the extension output shows HTML indicating I've hit a rate limit
2023-04-12 08:53:02.396 [info] Logging in for the following scopes: repo workflow
2023-04-12 08:53:02.397 [info] Trying without local server... (repo workflow)
2023-04-12 08:53:03.582 [info] Exchanging code for token...
2023-04-12 08:53:03.937 [error] GitHubTokenExchangeError: <!DOCTYPE html>
...

You have exceeded a secondary rate limit.

at t.GitHubServer.u (c:\Users\...\github-authentication\dist\extension.js:2:1018998)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async c:\Users\...\github-authentication\dist\extension.js:2:1013850

@rlaveycal Could you please try this again and see if it is a transient issue or not and let me know? Thanks!

Yes it's transient. I have logged in successfully. It's annoying however, and it would be nice if the error was properly surfaced rather than having to trawl the logs.

@inomdzhon - I'm sorry to hear that it isn't working for you still.
Could you please retry the steps with a different GitHub repo and see if that works? I wonder if there is something specific about the repo you are trying to open that is causing this issue? If that works then we could compare the repos and see if there is something specific about that repo that may be a problem.  Thank you so much for your help in debugging this with us!

I'm going to go ahead and close this issue since I don't see any new comments or pending issues and I've had the troubleshooting steps posted for a week since the bug fix went in.
If anyone in the future runs into issues, please try these troubleshooting steps before opening a new issue.

https://github.com/github/vscode-github-actions/issues/77#issuecomment-1503960028

Thank you all!

@KetchupOnMyKetchup thanks for instructions, but it doesn't fix problem :(
I'm using 0.25.4 and Code v1.77.3. There is Github Actions output on the bottom:
Activating GitHub Actions extension...

Cannot fetch github context
Error: Cannot fetch github context
	at Ao (/Users/i.mirdzhamolov/.vscode/extensions/github.vscode-github-actions-0.25.4/dist/extension-node.js:2:1597413)
	at async /Users/i.mirdzhamolov/.vscode/extensions/github.vscode-github-actions-0.25.4/dist/extension-node.js:2:1623054
	at async Rs (/Users/i.mirdzhamolov/.vscode/extensions/github.vscode-github-actions-0.25.4/dist/extension-node.js:2:1622125)
	at async E.n (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:96:6646)
	at async E.m (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:96:6609)
	at async E.l (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:96:6066)

Hello, I am facing the similar issues. I tried with all kinds of repos. Even I tried to login by creating a blank repo but still, I am getting the similar issue mentioned by @inomdzhon.
I am using VSCode Version v1.78 and github actions version v0.25.7.
Can anyone help me out on this?

Same issue:
Cannot fetch github context
Error: Cannot fetch github context
	at Ao (/home/nz/.vscode-server/extensions/github.vscode-github-actions-0.25.7/dist/extension-node.js:2:1597788)
	at processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async /home/nz/.vscode-server/extensions/github.vscode-github-actions-0.25.7/dist/extension-node.js:2:1634497
	at async Rs (/home/nz/.vscode-server/extensions/github.vscode-github-actions-0.25.7/dist/extension-node.js:2:1634453)
	at async v.n (/home/nz/.vscode-server/bin/b3e4e68a0bc097f0ae7907b217c1119af9e03435/out/vs/workbench/api/node/extensionHostProcess.js:97:6644)
	at async v.m (/home/nz/.vscode-server/bin/b3e4e68a0bc097f0ae7907b217c1119af9e03435/out/vs/workbench/api/node/extensionHostProcess.js:97:6607)
	at async v.l (/home/nz/.vscode-server/bin/b3e4e68a0bc097f0ae7907b217c1119af9e03435/out/vs/workbench/api/node/extensionHostProcess.js:97:6064)

Is there something specific about the computer that it is being tried on (blocked by a network, other VS Code extensions affecting it)? Would it be possible to try it on another machine?
Also just a head's up, sometimes there has been some caching issues with getting the new extension version with bug fixes and a restart has helped some folks.

i have the same issue You have not yet finished authorizing this extension to use GitHub. Would you like to try a different way? (device code)

@Harshgupta30, @nicolasalt Try reinstalling the VSCode and the extension, it worked for me interestingly. Hope this approach helps you guys 😉

Re-installing didn't work, however then I went to Accounts and removed Github Actions access to my Github account. Then the extension re-requested access and started to work 🥳

same issue here

I ran into a similar issue with the same issue. I was getting:
Error getting GitHub context
Bad credentials

What worked for me was to click on manage trusted extensions, which can be found on the accounts menu (lower left-hand side), which brings up a Manage Trusted Extension toolbar at the top.

(sorry have my instance on the right, but by default, it would be on the left.
Next, I unclicked Github actions and then re-clicked it. The application made me go through the steps to re-authenticate and allow vscode access.

After that, my issue was resolved. Hope that helps someone 😊

I ran into a similar issue with the same issue. I was getting:
Error getting GitHub context
Bad credentials

What worked for me was to click on manage trusted extensions, which can be found on the accounts menu (lower left-hand side), which brings up a Manage Trusted Extension toolbar at the top.  (sorry have my instance on the right, but by default, it would be on the left.
Next, I unclicked Github actions and then re-clicked it. The application made me go through the steps to re-authenticate and allow vscode access.
After that, my issue was resolved. Hope that helps someone 😊

This solution worked for me as well.  Thanks!

Re-installing didn't work, however then I went to Accounts and removed Github Actions access to my Github account. Then the extension re-requested access and started to work 🥳

It worked bro
👍

If someone has this problem, try putting this on terminal: code --open-external=https://github.com/login

If someone has this problem, try putting this on terminal: code --open-external=https://github.com/login

This finally made my day. Behind a corp VPN.

Warning: 'open-external' is not in the list of known options, but still passed to Electron/Chromium.

I have faced the same issues, it's mostly because you're behind a proxy(or VPN) either by you or your ISP(internet service provider), just try different ISPs if available

I am able to login with v0.25.4 👍

Please help me.

Do any one knows if this issue have been solved on Mac OS ?
I'm facing the same issue and even after removing and reinstalling VSCode I'm looping with the same kind of message :
"You have not yet finished authorizing this extension to use GitHub. Would you like to try a different way? (local server)"
Thank you in advance.

authors: Harshgupta30 KetchupOnMyKetchup Messi002 MetRonnie OpenCode RD-1988 Sergio-bzh aksuharun barweiss45 benjaminmillhouse debanjanbasu fox-john grzegorzjudas inomdzhon jmsvarbas nicolasalt rlaveycal sampathNG shryay spandan1460

repo: github/vscode-github-actions


text:
班级任务测试任务数据异常

如图所示，词达人分数与用时挂钩
其实分数无所谓，但用时50秒还是太离谱了点，选择题限时不到10秒，补全也限时最多30秒，这个50s数据很明显是异常的

  如图所示，词达人分数与用时挂钩 其实分数无所谓，但用时50秒还是太离谱了点，选择题限时不到10秒，补全也限时最多30秒，这个50s数据很明显是异常的

不好意思,我看看

  如图所示，词达人分数与用时挂钩 其实分数无所谓，但用时50秒还是太离谱了点，选择题限时不到10秒，补全也限时最多30秒，这个50s数据很明显是异常的

不好意思,我看看

应该是后端更新了,之前是前端验证的

  如图所示，词达人分数与用时挂钩 其实分数无所谓，但用时50秒还是太离谱了点，选择题限时不到10秒，补全也限时最多30秒，这个50s数据很明显是异常的
实在不好意思之前代码没测试,现在已解决。感谢issue!!!

经测试分数已正常，但单个单词用时仍然是50s


text:
Course audit - April 2019

Course audit, April 2019

@hectorsector when the course audit is completed, can we filter-branch the commits so you or I are no longer the commit author so we stop getting pings when people work on the course?

I am happy to do this - @hectorsector is it OK if I change history so @beardofedu is no longer the author of the initial commit on all branches, then force push up to the template repo and re-sync it?

Great! I've gone through and filter-branched on all of the branches, force pushed 'em up, and resynced the course. I tested it to make sure it would still work, and all is good. @beardofedu you should be off the hook!

Closing since this is long past!

authors: beardofedu brianamarie hectorsector

repo: githubtraining/intro-to-html


text:
chore: remove delayedHideAppIcon()

It looks like the bug has been resolved; I'm running MacOS and I've had no issues with showDockIcon: false.
I think we can simply drop this function (I added showDockIcon: false to make it more explicit, even though it was already the default value)

Nice. Any hints in a changelog at when this may have been resolved?

Nice. Any hints in a changelog at when this may have been resolved?

Nope, nothing there.

authors: adufr setchy

repo: gitify-app/gitify


text:
In generic-api-key rule, ignore PublicKey and PublicKeyToken

Is your feature request related to a problem? Please describe.
Capturing some rough ideas about how to possibly improve the generic-api-key rule...
Currently the rule detects public tokens such as this:
PublicKeyToken=a88a5c672034fd89

That's because the regex sees "Token" and it doesn't look backwards to see "Public"
Describe the solution you'd like
We might able to solve this by simply adding a negative lookbehind to a couple places in the regex. For example:
(?i)(?:(?<!public)key|api|(?<!publickey)token|secret|client|passwd|password|auth|access)(?:[0-9a-z\-_\t .]{0,20})(?:[\s|']|[\s|"]){0,3}(?:=|>|:=|\|\|:|<=|=>|:)(?:'|\"|\s|=|\x60){0,5}([0-9a-z\-_.=]{10,150})(?:['|\"|\n|\r|\s|\x60|;]|$)

In the regex above, notice:

Negative lookbehind (?<!public) before key. This way it won't detect PublicKey
Negative lookbehind (?<!publickey) before token. This way it won't detect PublicKeyToken

Describe alternatives you've considered

Use gitleaks:allow comment
Use .gitleaksignore
Use allowlist stopwords for the secret itself

Additional context
N/A
cc @zricethezav

I have encountered this issue too:
This line from a web.config file is incorrectly flagged by generic-api-key as containing secrets:

When I run the scan with redact: false it displays "b77a5c561934e089" as secret.
Strangely enough it does not flag any of the other 59 occurrences of PublicKeyToken as a secret. Some examples of other lines (that are before the line shown above):
<sectionGroup name="system.web.webPages.razor" type="System.Web.WebPages.Razor.Configuration.RazorWebSectionGroup, System.Web.WebPages.Razor, Version=3.0.0.0, Culture=neutral, PublicKeyToken=31BF3856AD364E35">

</sectionGroup>

I initially posted this issue on https://github.com/JoostVoskuil/azure-devops-gitleaks

authors: frankvaneykelen-work weineran

repo: gitleaks/gitleaks


text:
Ledger-Cardano-App 6.0.3 is unavailable

Your script requires Cardano-App v6.0.3 or above to be installed on the Ledger for Catalyst-Voting-Registration. But unfortunately, I can't find version 6 using Experimental Features -> My Ledger provider -> Enable it and set it to 4.
Please tell me how to install Cardano-App v6, thanks!

Hi,
i guess you are refering to the 09a_catalystVote.sh script.
Ledger has removed the Cardano-App version 6.0.3 from the Provider 4, you can use version 5.0.1. For that i created the script 09a_catalystVoteF10.sh
Best regards,
Martin

Yes, we are talking about the 09a_catalystVote.sh script.
Unfortunately, the 09a_catalystVoteF10.sh script also gives the following error:

Oh, you wanna create a hw voting key. Thats not possible right now. Just simply generate a normal cli voting key please like:
09a_catalystVoteF10.sh new cli myvote
You don't need a hw based voting key to register your hw wallet with it.

Did it work?

I have a cardano-pool where the owner is 2 wallets:

cli, where a little ada
hw Ledger, where almost all of ada is

As far as I understand, in order to Catalyst vote with my pool pledge, I have to register from hw wallet, right?

I have a cardano-pool where the owner is 2 wallets:

cli, where a little ada
hw Ledger, where almost all of ada is

As far as I understand, in order to Catalyst vote with my pool pledge, I have to register from hw wallet, right?

You need to sign the registration with your hw wallet yes, but not generating also the voting key on the hw wallet.
So all you have to do is basically:

09a_catalystVoteF10.sh new cli myvote to generate a voting key on the cli
09a_catalystVoteF10.sh genmeta myvote hw-wallet-account rewardpaymentaddr to generate the registration cbor metadata and sign it with your hw wallet
01_sendLovelaces.sh mywallet mywallet min xxx.vote-registrationcbor to submit the cbor on chain
09a_catalystVoteF10.sh qrcode myvote 1234 to generate the qr code for the voting app.
after a while (~hours) you can check the registration via 09a_catalystVoteF10.sh query myvote

authors: gitmachtl ihorzenich

repo: gitmachtl/scripts


text:
[FEATURE REQUEST] Vocos for Bark

https://charactr-platform.github.io/vocos/
It looks like it might improve the audio quality for speech generations.

Honestly a much needed feature, i should actually finally implement it lol

@nekogecko2 It looks like the link is no longer available, do you have an alternative?

repo: gitmylo/audio-webui


text:
🛑 MSP-002 is down

In 5dc0f71, MSP-002 (https://msp-portal.gw002.oneitfarm.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MSP-002 is back up in 01d1031.

repo: gitsrc/upptime


text:
line break?

This is working well. Thanks. Quick question. How do force a new line with a plan text message. tried /n and a few other permutations.
Thanks,
Pete.

This is working well. Thanks. Quick question. How do force a new line with a plan text message. tried /n and a few other permutations.
Thanks,
Pete.

Please visit Home Assistant yaml documentation.

service: whatsapp.send_message
data:
  clientId: default
  to: ###@s.whatsapp.net
  body:
    text: |-
        Hi it's a simple text message

        It has two lines

this can be closed

authors: ear9mrn giuseppecastaldo glmnet

repo: giuseppecastaldo/ha-addons


text:
🛑 Datasette is down

In 007cdb7, Datasette (https://datasette.givefood.org.uk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Datasette is back up in b52279e.

repo: givefood/upptime


text:
Feat: Sum

Implement Sum operator.

Can I work on this issue?

Hey @Ephraim-nonso I'm sorry, that issue was opened by mistake. This operator was already implemented here. Feel free to take another one. Sorry for that 🙏

authors: Ephraim-nonso raphaelDkhn

repo: gizatechxyz/orion


text:
Only trim whitespace for html content

:disappointed: Third time's a charm?
Looks like I keep swinging and missing on this solution.  Turns out there were more use cases for the trimming than I was aware.  While the prior PR fixed scenarios where we have whitespace at the beginning of a line like so:
    <% items.each do |item| %>

...it doesn't account for whitespace at the beginning of a line that outputs actual content.  For example:
    <%= "test" %>

In the above scenario, Markdown would still interpret the 4 spaces at the beginning of the line and improperly modify the structure of the html.
As far as I can tell, we have 2 options.  Either (a) we have a separate renderer for markdown content vs. html content or (b) we only trim html content.
I've put together an implementation for solution (b).  Would love to hear your thoughts.
@gjtorikian

No worries! Thanks for the commitment to a fix!

authors: gjtorikian obrie

repo: gjtorikian/graphql-docs


text:
make all error in caffe

CXX src/caffe/layers/multibox_loss_layer.cpp
In file included from ./include/caffe/common.hpp:19:0,
from ./include/caffe/blob.hpp:8,
from ./include/caffe/layers/multibox_loss_layer.hpp:8,
from src/caffe/layers/multibox_loss_layer.cpp:6:
./include/caffe/util/device_alternate.hpp:15:36: error: no ‘void caffe::MultiBoxLossLayer::Forward_gpu(const std::vector<caffe::Blob>&, const std::vector<caffe::Blob>&)’ member function declared in class ‘caffe::MultiBoxLossLayer’
const vector<Blob>& top) { NO_GPU; }
^
src/caffe/layers/multibox_loss_layer.cpp:574:1: note: in expansion of macro ‘STUB_GPU’
STUB_GPU(MultiBoxLossLayer);
^
./include/caffe/util/device_alternate.hpp:19:39: error: no ‘void caffe::MultiBoxLossLayer::Backward_gpu(const std::vector<caffe::Blob>&, const std::vector&, const std::vector<caffe::Blob>&)’ member function declared in class ‘caffe::MultiBoxLossLayer’
const vector<Blob>& bottom) { NO_GPU; }
^
src/caffe/layers/multibox_loss_layer.cpp:574:1: note: in expansion of macro ‘STUB_GPU’
STUB_GPU(MultiBoxLossLayer);
^
make: *** [.build_release/src/caffe/layers/multibox_loss_layer.o] Error 1

am using in cpu mode .
i make cpu=1
and use_opencv=1
in make file.

repo: gkioxari/RstarCNN


text:
Replace destructuring assignment

Correct fix for https://github.com/gl-vis/regl-line2d/issues/32 based on https://github.com/gl-vis/regl-scatter2d/commit/734e37241092ca4a61aa966c4a1649c13a1367fd

@archmoj what would you say?

Cool. Let's test this in plotly.js.
cc: @etpinard

Unfortunately the lines are disappeared after trying this PR:
https://circleci.com/workflow-run/b1163950-30c4-437d-91df-88972bd2eb0e

authors: abraham archmoj dy


text:
Provide links to toml.io

Pretty subjective. I can understand if you consider it overkill. Especially with the repetition but I kind of think that within reason the more explanation & context the better.

To make it easier for beginner's to understand the reference.
Based in the fact that tables in toml can be flattened out to multiple
lines of variables and given that that syntax is used in some gleam
projects, it feels like it makes sense to draw attention to the
potential representation.

Updated.

repo: gleam-lang/website


text:
Fail to uncompress zip file of 111 Mb

I am reading data from an ftp stream using ssh2 (The entire code is not shown)
var readStream = sftp.createReadStream("/dir/filename.zip") readStream.pipe(unzip.Parse())           .on('entry', entry => {             console.log(Inside transform callback....);             s3Manipulator.writeStreamToBucket('bucket_nm', entry.path, entry)           })           .on('error', err => {             console.log(Error thrown streaming ${err});           });
I am running this from an ubuntu system it shows upload activity, but after a while the error callback is invoked. The error is given below:
code:"Z_DATA_ERROR" errno:-3 message:"invalid literal/lengths set" stack:"Error: invalid literal/lengths set\n    at InflateRaw.zlibOnError (zlib.js:153:15)" __proto__:Object {constructor: , name: "Error", message: "", …}
I have no clue as to why this happens.
Any help is appreciated, thanks in advance.

events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: unexpected end of file
    at Zlib._handle.onerror (zlib.js:370:17)

similar problem, but with above error.
my zip file is above 200MB, if i use small ones, it;s ok.

@qburst-akhil I'm using decompress module now, ant it's fine.

authors: bi-kai qburst-akhil


text:
🛑 Schoelcher is down

In 435d1e2, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 22029a1.

repo: glefait/martinique-public-websites


text:
🛑 Schoelcher is down

In 85acfff, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in ebd17ae.


text:
🛑 Schoelcher is down

In 53374b4, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in 654e608 after 494 days, 16 hours, 35 minutes.


text:
🛑 Grande Baie Martinique is down

In 0fa8b8f, Grande Baie Martinique (https://grandebaiemartinique.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Grande Baie Martinique is back up in ef426e9.


text:
🛑 Schoelcher is down

In 89510e3, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in c86ac0c.


text:
🛑 Schoelcher is down

In eea27b9, Schoelcher (https://www.mairie-schoelcher.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Schoelcher is back up in b0181d5.


text:
🛑 Le Lamentin is down

In b93ec89, Le Lamentin (http://www.mairie-lelamentin.fr) was down:

HTTP code: 503
Response time: 968 ms

Resolved: Le Lamentin is back up in e6a4352.


text:
Highlights for code actions

Iam configuring my own colors following the highlight.lua file for a colorscheme. Everything was fine but now Iam unable to find a highlight group for most of the elements in code actions popup window and diagnostics window. Here is an image:

I think its quite clear what I would like to change. Is it possible to change the colors and if so, how?

message text is DiagnsoticText other highlight color in ui.colors . and there will change some ui in next version


text:
help

i have no idea how to install this in my game, please help

You don't "install" it in your game. It is a separate, standalone application that helps your mouse-related actions in the game.

repo: globalpolicy/pubgmobilemousehelper


text:
Fix erroneous use of dependent refresh tokens

The previous code (here, almost entirely removed) requested dependent refresh tokens unconditionally. These were then stored in a 47 hour cache, and there was never an update to the cache to refresh these credentials. Access tokens last for 48 hours, meaning that with the prior cache parameters, it was not realistically possible to reach the point of requiring a refresh via expiration. (A refresh triggered by token revocation would fail, and therefore is not worthy of special attention.)
Similar to the impossibility of a normal refresh, the timing requirement expressed by the required_authorizer_expiration_time parameter was more or less impossible to trigger in normal usage. With the default value of 60 seconds, the only way it could fire requires an hour to pass between the request being received and the dependent token for that request being retrieved. Therefore, this parameter is no longer checked, and emits a DeprecationWarning if used explicitly.

Test case for the deprecation added.

I rebased to clear the conflict -- it should build clean (passes for me locally), but let's watch CI.

repo: globus/action-provider-tools


text:
Option to use sass-embedded

In package.json you are depending on sass
https://github.com/glromeo/esbuild-sass-plugin/blob/main/package.json#L59
If I understand this correctly this is the dart-sass complier compiled into javascript. I wonder if there is any reason why you don't use sass-embedded instead https://www.npmjs.com/package/sass-embedded. It includes the sass binary compiled from the same dart-sass project. It is up to 10 times faster, mostly faster than libsass even.
I am looking at moving some scss files into esbuild but I don't want to wait for slow compilers when there are options :)
I would be okay we leaving the default sass but I want an option to go for sass-embedded.

It is up to 10 times faster, mostly faster than libsass even.

That statement might be true but the sass embedded is greatly penalised by being called repeatedly during a build so end up being slower because the integration overhead is the bottleneck (and I say that because I did a spike some time ago and the results were pretty poor).
To make it optional would pollute the plugin so I am not interested in going down this route

Interesting, I would be very interested in seeing these benchmarks and maybe bring up this up as an issue to the dart-sass team, if they did not already know it. (Sorry for opening the issue and not searching through closed issues, sorry.) I can see that it might be slower to convert small files with sass-embedded, but in our application we bundle all of our assets into 3 different entripoints, so the story might be a bit different on out side.
Althoug if you sent time invertigating this years ago I get it that it might not be so interesting looking into again.
Maybe you can add sentence onder Benchmarks in the readme?

I suggest you try esbuild-plugin-sass or try and fork this plugin and replace sass with the embedded.
Best of luck

authors: glromeo stoivo

repo: glromeo/esbuild-sass-plugin


text:
Issue with Linux: "Failed to merge configuration files"

When trying to run ILRepack against a .NET Framework application on Linux, I get an error which says that it failed to merge configuration files.
Repro:

Clone https://github.com/xunit/xunit
Run git checkout 9195c0b (this is on an in-progress branch, so I'm sending you to a moment in time)
Run dotnet build src/xunit.v3.runner.console

Working on Windows:

Broken on Linux:

Our CI process uses the Linux build for making the official builds.
It does appear to be successfully running the repack, because it generates an executable which appears to run okay:

I'm not sure what it's trying to do with the configuration file(s), but on the successful Windows side, the resulting "merged" .exe.config file is identical to original .exe.config file, except that the XML declaration has had encoding="utf-8" removed (why?) and standalone="yes" added (understandable). I would be happy to run ILRepack in such a way as to bypass .config file handling, since it would be just as easy for me to copy the original .exe.config file into the merged folder, but I didn't see any command line switch that would turn off config file processing.
I am invoking ILRepack via the executable:

dotnet --info on Windows:
.NET SDK (reflecting any global.json):
 Version:   5.0.103
 Commit:    72dec52dbd

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19042
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\5.0.103\

Host (useful for support):
  Version: 5.0.3
  Commit:  c636bbdc8a

.NET SDKs installed:
  3.1.406 [C:\Program Files\dotnet\sdk]
  5.0.103 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.All 2.1.25 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.25 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.25 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.12 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

dotnet --info on Linux:
.NET SDK (reflecting any global.json):
 Version:   5.0.103
 Commit:    9effbc8ad5

Runtime Environment:
 OS Name:     ubuntu
 OS Version:  18.04
 OS Platform: Linux
 RID:         ubuntu.18.04-x64
 Base Path:   /usr/share/dotnet/sdk/5.0.103/

Host (useful for support):
  Version: 5.0.3
  Commit:  eae88cc11b

.NET SDKs installed:
  2.1.813 [/usr/share/dotnet/sdk]
  3.1.406 [/usr/share/dotnet/sdk]
  5.0.103 [/usr/share/dotnet/sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.All 2.1.25 [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.25 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.12 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.3 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.25 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.12 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.3 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

For the moment I've worked around the issue by copying all the to-be-merged files into a new premerge folder so that there's no .config files hanging around to trip the issue: https://github.com/xunit/xunit/blob/ea3007119bff2358873ac6fe0d922a60c5090f82/src/Directory.Build.targets#L11-L56
Will still appreciate a fix, though, so I can stop the unnecessary file copying.

This issue is really quite frustrating.
I encountered that before.
Maybe it is possible to add an option to merge the config file or not.
This is not so difficult to implement.

Thanks @wmjordan for submitting the PR #291, the latest version has this option now: /skipconfig
Leaving the issue open to debug the underlying problem.

Also the logic to merge config files was rewritten using XLinq, so it's likely it has been fixed accidentally, worth a check. Might not even need the /skipconfig now.

Try https://www.nuget.org/packages/ILRepack/2.0.21, with and without /skipconfig

I just installed 2.0.21 and tried it on one of my production project, which was well merged by 2.0.15 before.
I got the following exception:

System.Runtime.Serialization.SerializationException: Deserializing object failed. ---> System.Xml.XmlException: Invalid data on root element, line 1, position 1.
at System.Xml.XmlExceptionHelper.ThrowXmlException(XmlDictionaryReader reader, String res, String arg1, String arg2, String arg3)
at System.Xml.XmlUTF8TextReader.Read()
at System.Xml.XmlBaseReader.IsStartElement()
at System.Xml.XmlBaseReader.IsStartElement(XmlDictionaryString localName, XmlDictionaryString namespaceUri)
at System.Runtime.Serialization.XmlObjectSerializer.IsRootElement(XmlReaderDelegator reader, DataContract contract, XmlDictionaryString name, XmlDictionaryString ns)
at System.Runtime.Serialization.DataContractSerializer.InternalReadObject(XmlReaderDelegator xmlReader, Boolean verifyObjectName, DataContractResolver dataContractResolver)
at System.Runtime.Serialization.XmlObjectSerializer.ReadObjectHandleExceptions(XmlReaderDelegator reader, Boolean verifyObjectName, DataContractResolver dataContractResolver)
at System.Runtime.Serialization.XmlObjectSerializer.ReadObjectHandleExceptions(XmlReaderDelegator reader, Boolean verifyObjectName, DataContractResolver dataContractResolver)
at System.Runtime.Serialization.XmlObjectSerializer.ReadObject(XmlDictionaryReader reader)
at ILRepacking.Steps.ResourcesRepackStep.GetRepackListFromStream(Stream stream)
at ILRepacking.Steps.ResourcesRepackStep.Perform()
at ILRepacking.ILRepack.Repack()
at ILRepacking.Application.Main(String[] args)

For my side, I can verify that the merge failure is gone now, using 2.0.21. I have been able to remove my premerge steps, and I verified that the only difference between the .config file in the unmerged folder and the .config file in the merged folder is that the merged version (benignly) added a UTF-8 BOM.
Change set (on top of the existing 2.0.21 upgrade in the commit prior): https://github.com/xunit/xunit/commit/920e3e521904741ffd2c1c100ef2073bbfa45cf5

@wmjordan thanks for trying it out!
I found the regression and fixed in https://github.com/gluck/il-repack/commit/01b54ff113123da84d8283df0b70175549e926b0
Will publish a new release soon, need to do some testing and fix a couple unrelated bugs.

I've now published https://www.nuget.org/packages/ILRepack/2.0.22
@wmjordan if you could try it out whenever you get a chance
@bradwilson you should now be able to call dotnet ILRepack.exe instead of mono ILRepack.exe on Unix, hopefully it'll work with both runtimes.

I'll be closing this issue but please feel free to file new bugs if something else is wrong.

Hi, I downloaded the new version, used it on the previously reported project and it was working very well now.

Verified it works here as well. Thanks!

authors: KirillOsenkov bradwilson wmjordan

repo: gluck/il-repack


text:
Do not "clean up" attributes when we copy attributes from the main assembly

In the case when we use the attributes from main target assembly we only want to remove redundant InternalsVisibleTo attributes and do not do general attributes clean up.
This breaks NHibernate build. We merge two .NET 2.0 assemblies (Antlr3 and ReMotion.Linq) into .NET 4 assembly (NHibernate). il-repack removes SecurityRulesAttribute from the output assembly as it do not match the .NET 2.0 assemblies (this attribute was introduced in .NET 4.0 and does not exist in .NET 2.0).
When the target assembly has AllowPartiallyTrustedCallersAttribute removing SecurityRulesAttribute from the output assemlby breaks ANY exception class (or ANY class which implements ISerializable) implemented in merged .NET 2.0 assembly as these classes do not have required [SecurityCritical] on GetObjectData method.
As a workaround we switched to /attrs switch for now which forces the execution path where code does not attempt to "clean up" attributes.

@gluck amended.

Thanks


text:
[Data] add data type u8

Background
#623
Description

When calculating arithmetic operations on u8 and i8, both are converted to i64 and calculated.
For the remaining types, cast to the larger type.

Pull Request Test Coverage Report for Build 3028020715

1399 of 1589 (88.04%)  changed or added relevant lines in 15 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 97.782%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/data/bigdecimal_ext.rs
4
5
80.0%

core/src/translate/data_type.rs
0
1
0.0%

core/src/data/value/convert.rs
93
95
97.89%

core/src/data/key.rs
0
6
0.0%

core/src/data/value/mod.rs
2
14
14.29%

core/src/data/value/literal.rs
2
17
11.76%

core/src/data/interval/primitive.rs
0
18
0.0%

core/src/data/value/binary_op/i8.rs
70
88
79.55%

core/src/data/value/binary_op/i32.rs
22
67
32.84%

core/src/data/value/binary_op/u8.rs
837
909
92.08%

Files with Coverage Reduction
New Missed Lines
%

core/src/data/value/mod.rs
1
97.42%

Totals

Change from base Build 3027836117:
-0.4%

Covered Lines:
34689

Relevant Lines:
35476

💛  - Coveralls

[comment]
Maybe we miss this
test!(Value::U8(2), Err(ValueError::ImpossibleCast.into()));

authors: 24seconds coveralls zmrdltl

repo: gluesql/gluesql


text:
Error on startup of first example

510  git clone https://github.com/gluon-framework/examples.git
511  ll
512  cd examples
513  ll
514  cd gluworld
515  npm install
516  node .
517  h
dean@pop-os ~/.../examples/gluworld (main) $ node .
[Gluon] starting browser...
file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/node_modules/@gluon-framework/gluon/src/index.js:198
const [ browserPath, browserName ] = await findBrowserPath(forceBrowser, forceEngine);
^
TypeError: (intermediate value) is not iterable
at startBrowser (file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/node_modules/@gluon-framework/gluon/src/index.js:198:40)
at async Module.open (file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/node_modules/@gluon-framework/gluon/src/index.js:255:19)
at async file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/index.js:17:3
Node.js v18.12.1
dean@pop-os ~/.../examples/gluworld (main) $
Happy to help any way I can.

What browsers do you have installed on your system?

chrome and firefox

I'm using !Pop_os so it's a flatpak install

I can run it from the command line with this command:
flatpak run com.google.Chrome

of course, I can also use
/usr/bin/flatpak run com.google.Chrome

authors: CanadaHonk witcradg

repo: gluon-framework/gluon


text:
Consider an option to serialize and hash separately

Background: https://github.com/richfitz/storr/issues/82, https://github.com/eddelbuettel/digest/issues/59#issuecomment-281145174, https://github.com/eddelbuettel/digest/issues/59#issuecomment-415616641.
In storr, serialization and hashing happen separately, which makes sense because storr actually needs that serialized object to put in storage. So when I plug fastdigest straight into storr, the same object gets serialized twice. Benchmarks: https://github.com/ropensci/drake/issues/498#issuecomment-415999409.
For the sake of https://github.com/richfitz/storr/issues/82 and https://github.com/ropensci/drake/issues/498, would you be open to separating these steps in fastdigest? I think this may also potentially make #1 easier if you choose to pursue it.

Please see my comment here https://github.com/ropensci/drake/issues/498#issuecomment-416004727 and let me know if this is still something that that seems like it needs doing/would help. I must admit I'm not clear what exactly is being asked here given the realities i describe in that comment wrt how fastdigest behaves (and gets it's speed).
~G

authors: gmbecker wlandau

repo: gmbecker/fastdigest


text:
Widescreen Slides

Is it possible to render a slideshow 16:9 format please?

Yes, when the call to remark.create() can take an options object, which supports a ratio option, which is by default 4:3, that may be set to whichever ratio you'd like, e.g.:
var slideshow = remark.create({ ratio: '16:9' });

Wiki: https://github.com/gnab/remark/wiki/Configuration#general

Thanks.

authors: gnab marktyers

repo: gnab/remark


text:
Idea: Add a way to download/upload an entire folder

it will very nice if moderators can download an entire folder. It will help plugin-configurating much easier for anyone who didn't host the server

Sounds good

authors: ItsukaHiro gnmyt

repo: gnmyt/MCDash


text:
Bug: Signing tx with ledger returns 422

What it solves
Resolves #2446
How this PR fixes it
Ledger recently introduced support for eth_signTypedData, and the same case as eth_sign returns an invalid V value in the signature, it should equal 27 or 28, but 0 or 1 is returned.
Also, it fixes a problem with eth_sign, our current code had incorrect logic for adjusting V value if a message was signed with\x19Ethereum Signed Message:\n32 prefix. If a message is signed with such prefix, then the V value needs to be adjusted to be > 30 for proper validation on the contracts side. In our case, we always increased it.
I also had to add the setupTests.js file which defines window.crypto.getRandomValues, otherwise the tests would throw an error
How to test it
Test signing with metamask, ledger, trezor - all of them should work

@francovenica , @mikheevm , should we also check WC->Ledger Mobile app->Ledger?

@liliya-soroka yes, it would be good to check all wallets

I'm wondering if this is one those cases when using the core-sdk would save us the trouble.

I'm wondering if this is one those cases when using the core-sdk would save us the trouble.

Maybe :) I actually copied signature validation logic from the sdk, but it only supports eth_sign and doesn't export it

I'm having the same issue I had in Dani's ticket regarding signing with a ledger account imported into the MM extension. The issue is for both off-chain and on chain tx

I can sign with ledger connected natively no problem.
This issue I think is only for win10 users. We should ask somebody with iOS or Linux to give it a try.

I tested with macOS

@francovenica did you see https://support.ledger.com/hc/en-us/articles/360020669680-Solutions-for-the-Windows-10-security-pop-up

Yes, tried it.
My ledger is up to date, the ethereum app as well. Ignoring the windows popUp cannot be done since is the one blocking the signature to actually go through.
I had this issue with ledger connected directly through the onboard option, but that was fixed with Dani's ticket. But the Import to MM is still an issue for me, even after Dani's fixes

I can't try brave because I have to import my hardware wallet into the MM extension there and when I try to do that I have that same warning sign from windows.

I asked Mati who also is on macOS and has a ledger to give it a try and he was successful as well
Tx 1087 on this safe
https://pr2472--safereact.review.gnosisdev.com/rinkeby/app/#/safes/0x9913B9180C20C6b0F21B6480c84422F6ebc4B808/transactions
I already checked MM, Trezor directly, trezor imported in MM, ledger directly, WC with the MM phone app. They all work

authors: francovenica katspaugh liliya-soroka mikheevm

repo: gnosis/safe-react


text:
(Hotfix) Safe Upgrade is not working

This fixes an overwritten parameter after merging #564

Travis automatic deployment:
https://pr582--safereact.review.gnosisdev.com

authors: fernandomg gnosis-info


text:
(Fix) #876 - Checksummed addresses

Closes #876
Description

Now when the addressBook loads, converts all the addresses entries to checksum version
When the safe loads, converts all the owner's addresses to checksum version
When replacing or creating an owner, checksums the address
I couldn't find any other case that remains uncovered, if you guys found another case in which an address could be entered non-checksumed, please let me know. Nowadays we also checksum custom token addresses

Travis automatic deployment:
https://pr884--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr884--safereact.review.gnosisdev.com/app

@Agupane
I had this issue twice, one when I was loading a safe for the first time and the other when I open the "Send Funds" form for the first time.
Idk if it related to this ticket, wonder if you can recognize the error in the console.
I haven't seen this issue in other PR's

For checksum
Every "Send" review form has the "Recipient" not checksumed when shown (Send funds, Send collectibles, Custom tx's). Now, they DO get checksumed when you check the Tx in the "Transaction tab", so is just during the review step.

Besides these the rest worked fine for me.

Travis automatic deployment:
https://pr884--safereact.review.gnosisdev.com/app

@Agupane
I had this issue twice, one when I was loading a safe for the first time and the other when I open the "Send Funds" form for the first time.
Idk if it related to this ticket, wonder if you can recognize the error in the console.
I haven't seen this issue in other PR's

For checksum
Every "Send" review form has the "Recipient" not checksumed when shown (Send funds, Send collectibles, Custom tx's). Now, they DO get checksumed when you check the Tx in the "Transaction tab", so is just during the review step.

Besides these the rest worked fine for me.

I couldn't reproduce the issue. I updated this branch with the current changes on development, maybe that could help to fix the issue you described @francovenica

Travis automatic deployment:
https://pr884--safereact.review.gnosisdev.com/app

Didn't see the error in the console anymore and the app didn't crash again.
The not checksumed address in the submit modal is not an issue according to the dev (since is only visual, the address is actually being checksumed when submited for the tx) so there is nothing to fix here.
PR approved

authors: Agupane francovenica gnosis-info


text:
TLS配置 Consumer配置是否支持通配

消费端访问不同的提供端,需要配置各自的证书私钥,在同一个项目下,往往各个微服务使用的是同一套证书,是否有通配功能,只需要配置一处就可以,而不是每个提供端都配置,这样配置文件有点冗余

https://go-chassis.readthedocs.io/en/latest/user-guides/tls.html
server name不填写就是全局的配置了

https://go-chassis.readthedocs.io/en/latest/user-guides/tls.html
server name不填写就是全局的配置了

这个是指的provider吧 consumer貌似不支持

可以支持rest.Consumer.xxx: xx
这样的配置，默认就读这个来加载client的tls配置
可否贡献这个特性

我们正在准备做这个事情    想法与您说的类似，想以rest.Consumer.xxx: xx 为默认配置
同时我们发现一个问题，当我们将consumer端的配置做成这样后，若当前服务同时配置provider和consumer时，会有key冲突
如下图

所以开了 这个 https://github.com/go-chassis/go-chassis/issues/984 issue来询问这个问题

repo: go-chassis/go-chassis


text:
+1, -1, and other reactions on issues and issue comments

There should be +1, -1, and other reactions on issues and issue comments just like GitHub and GitLab.

@bkcsoft Why are you opposed to this?

@jcgruenhage Because I'm a :trollface: perhaps? I really have nothing against this. I think I even :+1:'d it on the original Gogs-issue 😉 (I pretty much abuse this feature at work, so I'm all for having this in Gitea as well)

I hoped so, wasn't sure though, so I thought I might ask ^^

Fixed with #2856

authors: bkcsoft geek1011 jcgruenhage lafriks

repo: go-gitea/gitea


text:
Proposal: use https://github.com/src-d/go-git

Upside:

No dependency on particular versions of the on-site git command.
Better performance due to less fork/exec/IPC (measurement needed).
More reliable error handling (no stdout/stderr parsing).
Less complexities. (Pure Go function calls vs exec.Command + exec.Start + exec.Wait + stdout/stderr parsing).

If the is desired, then incrementally replacing some read-only Git operations will be a good start.

If I read correctly the code in its current version does not support merges ?
See https://github.com/src-d/go-git
Also, are you suggesting to use this as one of the implementations of the go-gitea/git interface as proposed somewhere else or as the primary API for git work ?
Did you test performance compared to libgit ?

Duplicate of #481. Please continue there

authors: bkcsoft strk typeless


text:
Reduce palette size of themes

Feature Description
Right now, the _base.less file defines all of the colours for the base theme (which is overridden for the dark theme), and the current palette size is… 121 colours, with 105 unique colours; some are duplicated but not referencing each other.
Honestly, I'd say it's a big stretch that this should be larger than 20, but the exact number isn't nearly as important. We definitely have too many.
Auditing things like colour contrast is basically not possible with this many combinations, and if someone wants to make simple changes (e.g. changing just the accent colour), this isn't really feasible, since there are so many different shades of each colour.
I think it's extremely worthwhile seeing what can be trimmed from the palette by either using a similar shade or opacity to blend with the background (in cases like the heatmap).
Screenshots
No response

I agree with needlessly eliminating variables but disagree completely with the required number of shades. At a minimum, since the stylesheet is using LESS, it should be computing required shades if necessary rather than hard-coding them, since it makes it difficult to discern what would need to be changed in case someone wanted to modify the colours.

since the stylesheet is using LESS, it should be computing required shades if necessary rather than hard-coding them

Computing colors in LESS is something I certainly want to avoid because those variables can not be interacted with at runtime (we already do that as part of the monaco themeing) and it makes 3rd party theming a lot more complex as they'd need to depend on our LESS sources. Imho, variables inside the styling must strictly stay custom properties.
CSS custom properties integrate much better than preprocessor variables. It's a shame that standard CSS still does not offer a way to modify colors at runtime, but there have been some recent proposals for just that in W3C, so maybe we'll get something in 5-10 years.

I think you're misunderstanding what I'm saying-- take the existing CSS variable definitions and replace the actual colours with versions computed using LESS functions. If you reference the compiled CSS, you'll still have static colours, but if you reference the LESS, you'll know where the colours came from.

We've migrated from Less to CSS now. Once color-mix is well supported in browsers, it can be used to reduce the number of colors, but I take it will be a year or two until that point.

This still doesn't respond at all to the argument that the stylesheet could due with substantially fewer shades than it has.
If we're talking about super-granular case like the heatmap in user profiles, this could easily be accomplished today with opacity instead of explicitly setting the background to a mixed colour. In all other cases, I still strongly disagree that a large number of shades are needed for standard UI elements; even a user with very good vision would be hard-pressed to identify ten different shades of a colour on all the standard UI elements, and much more than four is really pushing it.
I mentioned using LESS as a starting point for this issue, not the ending point.

I agree that there are too many colors. Is there a feasible plan about how to cleaning them up?

And actually, I have slight objection for such UI ( #22864) , I am not a fan of making UI too colorful .....
It's better to have some widely-accepted UI design patterns.

Another big reason against those kinds of changes is that the contrast tends to be rather poor. For example, the contrast between the red and green in the bar next to "README.md" there is effectively zero if you're red-green colourblind, and the contrast between the yellow text and the white background is also very poor.
This is also the main reason why I disagree with so many shades of colours. For example, the background colour of the header for the file element is #f7f7f7, and the colour of the main background is #ffffff. This is such a low contrast that it's effectively visible even for someone with a good monitor and good vision. This colour should be reduced to a darker shade or replaced with the original colour.

Luckily, the green-red gauge only provides some minor information, it only makes the UI looks slightly beautiful.
The real problem for me here is the orange color, I am the poor man who doesn't have a good monitor / good vision .... it's difficult for me to read how many files get changed at first glance .... 🤔

@clarfonthey Now there are new requirements for colors (focus/active), do you have more ideas about it? #24507

These are indiscussable I'd say. Fomantic UI has 4 different colors for button states, if we don't overwrite all, inconsistent colors will show up on buttons.

The discussion is about whether different states should be different colours, not whether we should set the colours in the CSS. I think we're both with you on making sure they actually follow the theme correctly, but the question is what the theme should be in the first place.

Interesting tidbit, number of color variables comparing GitHub and Gitea. GitHub has major spaghetti going on in their CSS variables, we are not nearly at that point, thankfully.
GitHub
Object.entries(getComputedStyle(document.documentElement)).filter(e => e[1].startsWith("--") && e[1].includes("color")).length
489

Gitea
> Object.entries(getComputedStyle(document.documentElement)).filter(e => e[1].startsWith("--") && e[1].includes("color")).length
224

Will be fixed with https://github.com/go-gitea/gitea/issues/19353 which will give us total color freedom.

authors: clarfonthey silverwind wxiaoguang


text:
latest 1.20.6 version (release yesterday) is older than 1.21.1

Description
I use an automatic update script to maintain the gitea service.
The latest version was 1.21.1 until yesterday where a 1.20.6 version was released.
https://github.com/go-gitea/gitea/releases
It breaks the database with error log :
Your database (migration version: 280) is for a newer Gitea, you can not use the newer database for this old Gitea release (260).
I think there is a version number error : it should be 1.21.2, no ?
Gitea Version
1.21.1
Can you reproduce the bug on the Gitea demo site?
No
Operating System
Debian
How are you running Gitea?
I use the https://github.com/go-gitea/gitea/releases repository
Database
MySQL/MariaDB

No. We released 1.20.6 because there is a security fix so the user still using 1.20.x should update their instance.  You can check the changelog.
For your case, you can skip this version because the same fix applies to 1.21.1 too.

Sorry. GitHub release will auto-set the newest one as the latest. We will change that later.
ref: https://github.com/go-gitea/gitea/issues/28291#issuecomment-1831684546


text:
Default to showing closed Issues/PR list when there are only closed issues/PRs

as title - remove one unessesary click sometimes ;)

ping

ready to 🚀


text:
Add MySQL FAQ section

MySQL is the ultimate root cause of repeated issues on here. We have the gitea convert command for some of the most common problems, however we should add a section to the FAQ to cover some of the most glaring repeated problems:
Signed-off-by: Andrew Thornton art27@cantab.net

Codecov Report

Merging #12780 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12780      +/-   ##
==========================================
- Coverage   43.26%   43.23%   -0.03%
==========================================
  Files         649      649
  Lines       72046    72046
==========================================
- Hits        31169    31152      -17
- Misses      35838    35849      +11
- Partials     5039     5045       +6

Impacted Files
Coverage Δ

modules/notification/mail/mail.go
34.48% <0.00%> (-3.45%)
:arrow_down:

modules/notification/ui/ui.go
80.55% <0.00%> (-2.78%)
:arrow_down:

modules/process/manager.go
72.50% <0.00%> (-2.50%)
:arrow_down:

services/pull/check.go
47.69% <0.00%> (-2.31%)
:arrow_down:

modules/git/blame.go
75.43% <0.00%> (-1.76%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
53.84% <0.00%> (-1.54%)
:arrow_down:

modules/log/event.go
57.54% <0.00%> (-0.95%)
:arrow_down:

models/notification.go
66.13% <0.00%> (-0.92%)
:arrow_down:

models/error.go
34.81% <0.00%> (-0.86%)
:arrow_down:

services/pull/pull.go
41.57% <0.00%> (-0.47%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 090024d...ef94e21. Read the comment docs.

authors: codecov-commenter zeripath


text:
Remove obsolete change of email on profile page

The change email on the account profile page is out-of-date
and unnecessary.
Changing email should be done using the account page.
Fix #13336
Signed-off-by: Andrew Thornton art27@cantab.net

Another option would be for this page to replicate the options on the account page.

🚀

Codecov Report

Merging #13341 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13341      +/-   ##
==========================================
- Coverage   42.12%   42.09%   -0.03%
==========================================
  Files         689      689
  Lines       75846    75845       -1
==========================================
- Hits        31949    31930      -19
- Misses      38664    38679      +15
- Partials     5233     5236       +3

Impacted Files
Coverage Δ

modules/auth/user_form.go
39.65% <ø> (ø)

routers/user/setting/profile.go
39.88% <ø> (-0.35%)
:arrow_down:

modules/indexer/stats/db.go
52.17% <0.00%> (-17.40%)
:arrow_down:

modules/git/utils.go
73.77% <0.00%> (-3.28%)
:arrow_down:

services/pull/temp_repo.go
26.59% <0.00%> (-3.20%)
:arrow_down:

services/pull/check.go
48.90% <0.00%> (-2.92%)
:arrow_down:

services/pull/patch.go
53.97% <0.00%> (-1.71%)
:arrow_down:

modules/queue/workerpool.go
58.77% <0.00%> (-1.23%)
:arrow_down:

modules/git/repo.go
46.19% <0.00%> (-0.51%)
:arrow_down:

modules/process/manager.go
75.00% <0.00%> (+2.50%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b64978d...4c265a4. Read the comment docs.

authors: codecov-io lafriks zeripath


text:
Make test more robust

The LFS API integration test has recently been flaky (e.g. https://drone.gitea.io/go-gitea/gitea/2559/10), so this PR increases the allow duration to reduce spurious failures.

LGTM

LGTM

Codecov Report

Merging #3167 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3167      +/-   ##
==========================================
- Coverage   34.74%   34.73%   -0.01%
==========================================
  Files         276      276
  Lines       39945    39945
==========================================
- Hits        13878    13875       -3
- Misses      24073    24074       +1
- Partials     1994     1996       +2

Impacted Files
Coverage Δ

models/repo_indexer.go
50.97% <0%> (-0.98%)
:arrow_down:

models/repo.go
41.28% <0%> (-0.19%)
:arrow_down:

models/repo_list.go
67.18% <0%> (+1.56%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ad6333...fbd7647. Read the comment docs.

authors: codecov-io ethantkoenig lafriks lunny


text:
Updated the section "Installation from sourse"

Updated the section "Installation from Sourse" for a more understandable and clean build

Added Go installation
Build from under the user " git " cleaner

Codecov Report

Merging #4233 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4233   +/-   ##
=======================================
  Coverage   19.96%   19.96%
=======================================
  Files         153      153
  Lines       30501    30501
=======================================
  Hits         6091     6091
  Misses      23496    23496
  Partials      914      914

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ba165e...df7d802. Read the comment docs.

authors: Mauladen codecov-io


text:
Add changelog for 1.5.0-RC1

As title
~Blocked by #4323~

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a711d41). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4324   +/-   ##
=========================================
  Coverage          ?   20.09%
=========================================
  Files             ?      153
  Lines             ?    30705
  Branches          ?        0
=========================================
  Hits              ?     6171
  Misses            ?    23590
  Partials          ?      944

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a711d41...ba9c339. Read the comment docs.

I think changelog should be only generated from label changloge and milstone 1.5.0 to keep it clean:
https://github.com/go-gitea/gitea/issues?q=label%3Achangelog+is%3Aclosed+milestone%3A1.5.0
If anything else from this needs to be included label shoud be added to that PR

@lunny conflict resolved
@daviian / @lafriks thanks for feedback. I've changed the list to only include the items with changelog label

It is probably worth to review if there is anything else that is worth to mention in there and than label should be added to that issue

@lafriks, may be sorting enhancement https://github.com/go-gitea/gitea/pull/3969

@axifive added.
@lafriks / @daviian please review changes

@daviian need your approval

We can also add #4188 to indicate that binary can be check with the public key. We could also indicate the public key fingerprint.

There are many more security fixes that didn't find it's way to a minor version of 1.4
These should be included as well.
Also I think there are further noticable PR's.
See https://github.com/go-gitea/gitea/pulls?page=2&q=is%3Aclosed+milestone%3A1.5.0+is%3Apr+-label%3Akind%2Fdocs++-label%3Abackport%2Fv1.4+-label%3Akind%2Fbug+-label%3Akind%2Ftranslation&utf8=✓
Of course not all of them, but some.

@daviian I added changelog label to 11 more PR's from that list, just need to add them also to this PR

@daviian @sapk done

authors: axifive codecov-io daviian lafriks sapk techknowlogick


text:
Does gorm 2.0 support annotation like sql:"index"?

Your Question
Does gorm 2.0 support annotation like sql:index ?
gorm还支持不支持 sql:"index"这种用法？
for example:
type Model struct {
	ID        uint       `gorm:"primary_key"`
	CreatedAt time.Time  `gorm:"not null" sql:"DEFAULT:current_timestamp"`
	UpdatedAt time.Time  `gorm:"not null" sql:"DEFAULT:current_timestamp"`
	DeletedAt *time.Time `gorm:"index:idx_deletedAt" sql:"index"`
}

Expected answer

offical answer is : tag must use gorm , does not support sql any more.

repo: go-gorm/gorm


text:
Upgrade 1.25.5->1.25.7: AutoMigrate returns the error "insufficient arguments"

GORM Playground Link

https://github.com/go-gorm/playground/pull/1
Description

I'm using the postgres backend and ran into this error. Updating gorm.io/driver/postgres from v1.5.4 to v1.5.7 seems to resolve the issue.

v1.5.7

gorm.io/driver/postgres v1.5.9
gorm.io/gorm v1.25.12
Here is my version, and it still has this error  pq: got 2 parameters but the statement requires 1

authors: ReeseWang todeadarius zieen


text:
over 60 "cluster slots" commands per second when using the ClusterClient type

makes the first node's cpu usage among the cluster containing 100 masters and 100 slaves becomes too high.
would you please give me some tips?

You can try https://github.com/go-redis/redis/pull/1391/files which picks random node for loading cluster slots.
cluster slots command is issued on every error except redis: nil (but not more often than once in 100ms). If you have lots of errors it can explain high number of cluster slots. I assumed it should be harmless.

authors: tancehao vmihailenco

repo: go-redis/redis


text:
Error github.com/go-redis/redis/internal/instruments.go:24:23: undefined: otel.Meter

Hello, I use go1.12.7 after using go get github.com/go-redis/redis or  go get github.com/go-redis/redis/v8 error:
Error github.com/go-redis/redis/internal/instruments.go:24:23: undefined: otel.Meter
How to fix it, thanks.

https://github.com/go-redis/redis/issues/1534


text:
Property collection should be collections in root path

http://geo.kralidis.ca/go-wfs/ currently returns a result with one of the properties as collection. This should be collections in conformance to the schema defined in the specification document.

Thanks Ahmed, fixed.  If you don't want to run your own, we've got an instance not too far behind at http://wfs.jivanamara.net

authors: JivanAmara ahmedOpeyemi

repo: go-spatial/go-wfs


text:
#2748 passing request context to ConextValidate()

Signed-off-by: Nick Dimov 3619341+dimovnike@users.noreply.github.com

Looks like there is a test that actually asserts the line you changed

Submitted a fix. The problem  is that i can not run tests locally, they fail even in master:
--- FAIL: TestClient (35.47s)
    --- FAIL: TestClient/generate_client (0.00s)
        --- FAIL: TestClient/generate_client/packages_mangling (4.87s)
            utils_test.go:30:
                        Error Trace:    utils_test.go:30
                                                                client_test.go:133
                                                                client_test.go:234
                                                                client_test.go:309
                        Error:          Expect "// Code generated by go-swagger; DO NOT EDIT.

                                        package client
...

hope this will work!

@dimovnike what are the errors you see when you build locally?  Can you share which OS + CPU architecture you are developing on?

@dimovnike what are the errors you see when you build locally? Can you share which OS + CPU architecture you are developing on?

the build is ok but go test is failing.  I use Ubuntu 20.04, and go version go1.16.7 linux/amd64.
I just found this in the error log:

note: module requires Go 1.17

its probably because of this. I will update golang and will try again.

authors: casualjim dimovnike

repo: go-swagger/go-swagger


text:
Fix runtime crash

Fixes #2

/cc @danderson

PTAL @josharian @danderson

Cool. Will submit. @josharian can school us if we're wrong. 😂

I've been pondering this intermittently. I don't understand the handling of refs. It's not actual refcounting. It seems to me e.g. that the finalizer could set refs = 1 before setting a new finalizer, which could speed up things considerably. Is a bool enough?

Yeah, while writing docstrings explaining this interplay, I realized that this is not actually refcounting, but something akin to mark-and-sweep. As currently written, the weakref will need one GC cycle per call to Get before the interned value is actually deleted. So, it does still save memory, but the freeing schedule is unpredictable.
I think Josh is right that in the "re-set finalizer" case, we can set refs=1 to speed things up. In that case, a dangling Value will get collected at most 3 GC cycles after its last reference was dropped. I'll send a patch for same alongside the docstrings explaining this. I welcome a bikeshed for a new name for refs, since it's not a refcount.

It seems to me e.g. that the finalizer could set refs = 1 before setting a new finalizer, which could speed up things considerably.

Yeah, that's a good idea.

Is a bool enough?

Empirically, yes. PR coming.

authors: bradfitz danderson josharian


text:
core/commands: add flags to 'urlstore add' - cid-version, hash, raw-leaves

This PR adds --cid-version, --hash, and --raw-leaves flag support to the urlstore add command.
Recreated since the rebase didn't update the last PR correctly.

https://github.com/ipfs/go-ipfs/issues/6065 is probably a better alternative to this.

Note: urlstore add requires raw leaves to work so it doesn't really make sense to make it optional.

repo: gochain-io/go-ipfs


text:
Merge into Godot Physics Engine

Merging into Godot Engine's Engine may be possible in near future because of Physics work.
This issue tracks the Godot Physics Engine merge situation.

Pausing because of the lack of movement.

repo: godot-extended-libraries/hydro


text:
C argument names that conflict with rust keywords are given a gd_ prefix.

Fixes: https://github.com/godot-rust/godot-rust/issues/753
This feature existed in previous iterations of godot-rust, but this patch improves the implementation to cover all known keywords as of today.

I did the linear search because I felt that the array was more human readable, but this alternative you suggested is readable enough.
I'll also make the suggested name change. I just didn't know that was already a common practice.

The change has been made. It plays nicer with cargofmt than I expected.
I should have just done this from the start.

Thank you! Two minor things:

Could you combine all the commits into a single one?
git commit --rebase -i might help (or many UIs offer a simplified "combine multiple commits to one" option).
If you need help, let me know, I'll gladly assist.
Run cargo fmt before committing, that way you make sure the CI passes.
Don't hesitate to use  #[rustfmt::skip] in this case (on the match or the entire function). Just make sure indentation/spaces etc are still correct.

Accidentally closed (I hate this GitHub feature), will need to check with the author to regain push rights.

Accidentally closed (I hate this GitHub feature), will need to check with the author to regain push rights.

Squashed the commits as discussed. Thanks a lot for your contribution! 🚀
bors r+

Squashed the commits as discussed. Thanks a lot for your contribution! 🚀
bors r+

authors: Bromeon IamTheCarl

repo: godot-rust/godot-rust


text:
Game crashes when changing scenes while using a SubViewport with main Viewport's 2D world in it and a sound playing

Tested versions
v4.3.stable.official [77dcf97d8]
System information
Godot v4.3.stable - Windows 11 (24H2, 26100.2605) - GLES3 (Compatibility) - NVIDIA GeForce RTX 4090 (NVIDIA; 32.0.15.6636) - 13th Gen Intel(R) Core(TM) i9-13900K (32 Threads)
Issue description
I'm sorry if I'm not reporting this correctly, this is my first time addressing such a complex issue and took a while to figure out the details.
The game crashes when switching scenes while using a SubViewport containing the main Viewport's World2D and a sound playing.
However, the crash does not occur if all audio nodes are removed from the scene that includes the SubViewport.
This issue seems to be caused by the engine not properly removing the SubViewport when switching scenes.
Crash cause

Call stack

Viewports related operations

P. S. The SubViewport in question isn't even an audio listener, so this is confusing. (Edit) Oooh... I guess it is, since it has main Viewport's world? Is this how it works?
Steps to reproduce

Set up a scene with a SubViewport containing the main Viewport's World2D.
Include audio node playing in this scene.
Switch scenes using either get_tree().change_scene_to_file() or get_tree().change_scene_to_packed()
(e.g., transitioning between a main menu and a gameplay scene)
Repeat the scene transitions back and forth.
The game crashes randomly after one or more scene switches.

Example
Set this in a scene you're switching to.
func _ready() -> void:
  $SubViewport.world_2d = get_tree().current_scene.get_viewport().world_2d

Then switch scenes with get_tree().change_scene_to_file() or get_tree().change_scene_to_packed() back and forth
(e.g. from main menu to gameplay scene and back) and the game randomly crashes at some point (after the first or a few scenes switches)
Manually introducing a custom removal mechanism for the SubViewport appears to resolve the crash.
func _ready() -> void:
  tree_exiting.connect(func(): $SubViewport.world_2d = null)
  $SubViewport.world_2d = get_tree().current_scene.get_viewport().world_2d

Minimal reproduction project (MRP)
subviewport-crash.zip

Could you check if you still see the issue in recent 4.4 dev builds?  This looks similar to https://github.com/godotengine/godot/issues/89212, which should be fixed by https://github.com/godotengine/godot/pull/91123.  MRP ran ok for me on a recent build from master, 4.3 stable crashed.

repo: godotengine/godot


text:
Error created from Template project page

Godot version:
v3.0.1.stable.official
OS/device including version:
Windows 10 64-bit.
Intel i7 7700k
GTX 1080
Issue description:
When browsing the Templates window the terminal produces errors when trying to load pictures from the test projects it is listing.
Error returned:
ERROR: Image::copy_internals_from: Condition ' p_image.is_null() ' is true.
   At: core\image.h:327
ERROR: _jpegd_mem_loader_func: Couldn't initialize ImageLoaderJPG with the given resource.
   At: modules\jpg\image_loader_jpegd.cpp:125
ERROR: _png_error_function: Not a PNG file
   At: drivers\png\image_loader_png.cpp:62
libpng error: Not a PNG file
ERROR: ImageLoaderPNG::_load_image: PNG Corrupted
   At: drivers\png\image_loader_png.cpp:94
ERROR: _load_mem_png: Condition ' err ' is true. returned: Ref<Image>()
   At: drivers\png\image_loader_png.cpp:252
ERROR: Image::copy_internals_from: Condition ' p_image.is_null() ' is true.
   At: core\image.h:327
ERROR: _jpegd_mem_loader_func: Couldn't initialize ImageLoaderJPG with the given resource.
   At: modules\jpg\image_loader_jpegd.cpp:125
ERROR: _png_error_function: Not a PNG file
   At: drivers\png\image_loader_png.cpp:62
libpng error: Not a PNG file
ERROR: ImageLoaderPNG::_load_image: PNG Corrupted
   At: drivers\png\image_loader_png.cpp:94
ERROR: _load_mem_png: Condition ' err ' is true. returned: Ref<Image>()
   At: drivers\png\image_loader_png.cpp:252
ERROR: Image::copy_internals_from: Condition ' p_image.is_null() ' is true.
   At: core\image.h:327
ERROR: _jpegd_mem_loader_func: Couldn't initialize ImageLoaderJPG with the given resource.
   At: modules\jpg\image_loader_jpegd.cpp:125

Steps to reproduce:
Download Godot from Steam.
Open Godot through Steam Launcher.
Click Template Tab.
Browse any page.
This ONLY seem to happen the first time each page is loaded, so after moving back to a page you have seen before it does not produce the error again.
Minimal reproduction project:
No project needed.

Download Linux version run and Open Asset Library, same thing happens

This seems to be largely cosmetic but it it's not pretty. This happens in 3.0-stable also but less frequently. I suspect some race with the download threads.

I reproduced this bug on Ubuntu Artful (17.10)
> uname -a
Linux y580 4.13.0-37-generic #42-Ubuntu SMP Wed Mar 7 14:13:23 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

authors: P1llus hpvb octavz roguh


text:
Update rate capped to 60 times per frame when going into fullscreen

Godot version 3.0.2
When I turn on fullscreen and enable the "print fps" in the project settings, the update rate seems to be capped at 60 times/second. Vsync is disabled in graphics settings and in project settings as well. Also happen when setting resolution to screen resolution and enabling "borderless"
Steps to reproduce:
-create empty project
-enbale "print fps" in project settings
-turn off vsync
-enable fullscreen

When you go into fullscreen mode, does it still print out a higher fps than 60?

Yes, my fps is above 5000 on fullscreen, but I just tested it on Linux, I need to test it on windows now.

Ok, so I tested it on linux and I have no issues whatsoever. High fps when going fullscreen; I have no idea why this is happening on my windows machine..

I tested on win10 here and I've no problem, probably it's something in your system.
Your machine is a notebook right? Maybe the problem could be something related with the optimus technology (maybe godot it's not using the nvidia). I don't have any hybrid GPU to test, so I can't help much.

@k4sma Did you try what the comment above said? (or you can try Godot 3.2.1 too)
I tested with GTX 1060 on Windows 10 and don't have this issue either.

I'll close this as nobody else has been able to reproduce this recently. Please comment (with the issue template filled) if you can still reproduce this.

authors: Calinou KoBeWi guilhermefelipecgs k4sma


text:
UWP build fails

Godot version:
8c9e10553cd429857086a9d635fc55305065ff76 (current)
OS/device including version:
Windows 10
Issue description:
I built 32 commits ago with "just" disabling websocket and upnp because something there crashed the build as well and i didn't necessarily need them for testing uwp ^^. However, after pulling it sais something about OSUWP, that it cannot be instanciated at /platform/uwp/app.cpp:102 because it would be an abstract class. I'm a Linux user, so i don't know how to copy&paste from these shells but i think that is the main info and it could be easy to reproduce.
Steps to reproduce:
If you have windows 10, build for uwp in it, if you don't have windows 10 i envy you a bit

it's because of this get_current_video_driver thing, sounds reasonable


text:
Shader error on AMD GPU's

Godot version:3.0.6
**OS/device including version:Windows 10 **
GPU: AMD R7 from an AMD A8 7650K APU
Issue description:
Whit the thread model set on Multi-Threaded on AMD GPU's after you will play any scene (3D or 2D) in the console you will see:
SCRIPT ERROR: (null): Expected 'shader_type' at the beginning of shader.
At: :1
ERROR: _update_shader: Condition ' err != OK ' is true.
At: drivers/gles3/rasterizer_storage_gles3.cpp:1697
NO CUSTOM SHADERS WERE USED
Steps to reproduce:
Create any scene (3D or 2D) put the thread model on Multi-Threaded on an AMD GPU and run it

https://github.com/godotengine/godot/issues/15998

authors: ComicoTeam VitaZheltyakov