Add Integration Tests

This is quite a big PR as it refactors a bunch of the test set-up to handle two different types of tests, as well as adds integration tests for tiny-invariant, rollup-plugin-postcss, styled-components, and regeneratorRuntime. Fixes #481
I initially tried to have the integration tests have their own package.json and node_modules, as the build-withConfig folder had before (and with a new CI workflow with working-directory set to the integration test dir), but this proved extremely confusing and very time-consuming to deal with.
It basically requires that scripts, like tests, run from the integration test dir, as if they run from root, they won't read the node_modules (node scans upward, so from the integration test dir, it'll get the integration deps and the TSDX deps, but the opposite doesn't work). And trying to run scripts from there causes lots of inconsistencies all over the place, so I just scrapped that idea.
Open to ideas on better integration testing structures, but the other option proved completely not worth the time, so don't want to spend a ton more time trying to get something else to work either.
Also found some integration failures all over the place while adding this, welp 😕 These include:

--extractErrors removes invariant but doesn't replace it with ErrorProd or ErrorDev. And it leaves the import in, just importing nothing 😕

warning isn't changed what-so-ever
I still think this should be split out as its own Rollup or Babel plugin, it's fairly independent of TSDX, doesn't seem to be used much by users, and can benefit more than just TSDX

rollup-plugin-postcss any PostCSS plugins like autoprefixer and cssnano work, but the name of the extracted CSS file is the same as the output, i.e. package-name.cjs.development.css
babel-plugin-styled-components doesn't work, plugin order matters for it. See #543

Ok fixing the conflicts with #582 in yarn.lock was not easy 😕 . But done now, so this is ready to merge. Should go in before #525 , so need to review this soon

Rebased with #621 and added execWithCache everywhere like it has. This actually took a bit of effort due to the number of commits and renames to rebase against.
Moved grep helper to utils/shell.js
Moved util to utils/fixture.js (also took some effort due to renames / rewriting imports)
Removed unused utils/psTree.js and its dependencies
Fixed a rebase issue I made where I accidentally merged two commits together that should be split
Added a test that ensures Babel presets are merged, which was the original goal of #481, to add a regression test for the fix in #473


Folds close automatically when leaving insert mode by pressing

Neovim version (nvim -v | head -n1)
Nvim 7
Operating system/version
Arch Linux
How to reproduce the issue

-- SETTINGS --
vim.o.foldenable = true
vim.o.foldlevel = 99 -- Using ufo provider need a large value, feel free to decrease the value
vim.o.fillchars = [[eob: ,fold: ,foldopen:,foldsep: ,foldclose:]]
vim.o.foldcolumn = "0" -- changes the width of the fold column (removes the numbers as well)
vim.o.foldlevelstart = -1

-- CONFIG https://github.com/kevinhwang91/nvim-ufo#advanced-configuration --
require("ufo").setup({
open_fold_hl_timeout = 100,
fold_virt_text_handler = handler, -- custom text next to fold
})

-- KEYMAPS --
vim.keymap.set("n", "zR", require("ufo").openAllFolds, { desc = "Open All Folds" })
vim.keymap.set("n", "zM", require("ufo").closeAllFolds, { desc = "Close All Folds" })
Expected behavior
Folds should maintain their state no matter what action I perform?
Actual behavior
In my regular javascript / typescript files, whenever I leave insert modus, or perform another action like for example delete word in normal mode, the folds automatically collapse.

Any idea what could trigger this?
I cleared the ~/.cache/nvim/ufo.log and watched for error but no errors are produced by this action.

setlocal foldlevel=99, switch buffer from the window with small foldlevel cause this issue.

So how do you suggest I can prevent this from happening?
If I manually enter :setlocal foldlevel=99, it does fix it for that buffer..
But it doesn't seem like a sustainable implementation if I have to manually enter that command every time I open a buffer? Can I change this in the configuration? I added this into my ufo.lua config. But it doesn't seem to solve the problem.
-- vim.o.foldlevel = 99 -- FIX: this bugs on javascript files, use the setlocal one below
vim.cmd("setlocal foldlevel=99")

I'm quite new at all this so it would be nice if you could send me in the right direction.
Thanks a lot!

You should find what script changes your foldlevel for javascript buffer.

Is an autocommand the solution? For example... but this interferes with a session manager that saves the fold inside a session. Each time we set the fold level locally the folds open up completely on buffer open.

Yes, BufEnter will make it work, but I suggest you set foldlevelstart = 99

Adding this comment if someone ends up with similar problem as me.
I had same issue caused by diffview.nvim which is setting foldlevel to 0 after opening DiffviewOpen or DiffviewFileHistory. In this case to fix this problem you can add these hooks into diffview.nivm config(also see this(#132) issue) from their repository
require("diffview").setup({
    enhanced_diff_hl = true,
    hooks = {
        diff_buf_read = function(bufnr)
            vim.opt_local.foldlevel = 99
            vim.opt_local.foldenable = false
        end,
        diff_buf_win_enter = function(bufnr)
            vim.opt_local.foldlevel = 99
            vim.opt_local.foldenable = false
        end
    }
})


Make marker message/label clickable?

I'm trying to make my marker popups clickable.  When I use html in the marker popup the html is rendered correctly but ng-click doesn't work.  Here's what I have.
 $scope.map.markers.push({
    lat: lat,
    lng: lng,
    message: '' + data.results[i].locationName + '',
    draggable: false
});

Anyone know what I'm doing wrong or know how to call a function when the marker popup is clicked?

This issue was moved to angular-ui/ui-leaflet#20

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.

I think this is easiest way:  http://stackoverflow.com/a/27935131/2742162


Flip ui beta flag to default true

This effectively makes what used to be the ui beta, the official ui.

Awesome!

Awesome!


composer-cli requires root on RHEL 8 but not on Fedora 31/32

While working on osbuild/ansible-osbuild#58, I found that running composer-cli with osbuild-composer on RHEL 8.2 requires sudo, but Fedora 31 and 32 allow the normal fedora user to use composer-cli to talk to osbuild-composer.
I'm not sure if this is by design, or if we have a bug.

Oops. Fixed in #674.

@larskarlitski Nicely done, sir!


rust: bump protobuf to 3.2.0

Hello! This branch updates protobuf to 3.2.0, which the latest version of the [protobuf]https://github.com/stepancheg/rust-protobuf/) crate.
Normally, I wouldn't open a PR to bump a dependency like this, but I'm in the process of updating a vendored rust-analyzer in a monorepo, and I can't have both protobuf-3.2.0 and protobuf-3.1.0 together without some patching that I'd strongly prefer to avoid. This was the approach of least resistance.
Test plan
Ran ./dev/proto-generate.sh; it appeared to have succeeded (and partially reformatted docs/scip.md file).

Apologies for missing this earlier. Feel free to @ of the maintainers directly in the future if your PR hasn't been reviewed/approved within a few days.

Apologies for missing this earlier.

No worries! I appreciate you landing this.

Feel free to @ of the maintainers directly in the future if your PR hasn't been reviewed/approved within a few days.

Gotcha, can do! Wasn't sure of the etiquette!


Added search box and functionality to ViewSelector

Added search box and functionality to ViewSelector so that views can now be searched for convenience.

Not sure what's going on with the pipelines, everything seems to have passed but it is hung somewhere, I'll force merge


Text / Design Issue - Inviting people to answer your question

Describe the bug
Heading says: People Asked
Description says: select people who you think might know the answer.
Heading is not machining to its description. Heading correction to "Invite people to Answer" or just "Invite people"

PS. Answer to a question and name of this site is Answer. It this case due to context, usually user consider this "Answer to a question"
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots or video to help explain your problem.
Platform

Device: [e.g. Desktop, Mobile]
OS: [e.g. macOS]
Browser and version: [e.g. Chrome, Safari]
Version: [e.g. v1.2.0]

Good observation here.
Since it's a guide for telling users that you can invite people in the community with relevant skills/experience to answer the question. It's a targeted outreach for faster answer.
Here's my version:
Title: Invite people to Answer/Get Expert Help/Ask the Community
Description: Think someone in the community might have the answer? Invite them to get the conversation started!

Text should be simple and direct. Update:

Title: Invite People
Description: Invite people you think can answer.

Text should be simple and direct. Update:

Title: Invite People
Description: Invite people you think can answer.

The description is concise, I agree, but is it possible to keep the title to: Ask the Community.
It sounds inclusive, encourages collaboration, and shows open nature.
Since we've already had Invite in the description, why not just keep one?
Just a personal view here.

@PrimmaAnna When you post a question, you are already "ask the Community".


Update dbt-cloud-tips.md

adding new shortcuts to tip and tricks page per slack
What are you changing in this pull request and why?

Checklist

[ ] Review the Content style guide and About versioning so my content adheres to these guidelines.
[ ] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding new pages (delete if not applicable):

[ ] Add page to website/sidebars.js
[ ] Provide a unique filename for the new page

Removing or renaming existing pages (delete if not applicable):

[ ] Remove page from website/sidebars.js
[ ] Add an entry website/static/_redirects
[ ] Ran link testing to update the links that point to the deleted page

ah great callout @benmosher ! i've made some tweaks to add windows shortcuts too!

thanks so much @thisispvb !


Financial Analysis output

Financial Analysis output
User Story
As a financial advisor,
I want to see the amounts formatted with 2 decimals and their currencies,
so I can see the final amounts more clearer in the Financial Analysis.
Acceptance Criteria

It's done when the total net amount outputs Total: $2,561,231.00.
It's done when the average total rate of change outputs Average Change: $2,561.00.
It's done when the greatest transaction outputs Greatest Increase in Profits: Feb-2017 ($2,561,231.00).
It's done when the lowest transaction outputs Greatest Decrease in Profits: Feb-2017 (-$1,561,231.00).

Final Output
    Financial Analysis
------------------------------------------------------------------------
    Total Months: 86
    Total: $38,382,578.00
    Average Change: $7,803.48
    Greatest Increase in Profits: Feb-2012 ($1,170,593.00)
    Greatest Decrease in Profits: Sep-2013 (-$1,196,225.00)
------------------------------------------------------------------------


update: remove experimental wording about middleware

What kind of changes does this PR include?

New or updated content

Description
This PR updates the middleware in preparation for the 2.6 release.
This PR must be merged after https://github.com/withastro/astro/pull/7109 is merged.
I am not really sure if the information I added is under the correct category; I would greatly appreciate some guidance!
Here's the list of the additional changes:

heading about type safety, which also talks about the utility defineMiddleware, which wasn't mentioned before;
updated the api-reference.mdx to add information about context.locals and Astro.locals.

as discussed "Middleware (experimental)" still in the heading, in the left menu.


Fix token account listing on Postgres

Postgres has much stricter constraints on DISTINCT/GROUP BY queries - specifically,
the fields used in ORDER BY must also be part of SELECT, and therefore may need
an aggregate function applied in order to give deterministic results.
Implement ordering by "seq" or "updated", which seem like the most useful.
Resolves #416

Codecov Report

Merging #422 (d894408) into main (f56d701) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #422   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          277       277
  Lines        14858     14860    +2
=========================================
+ Hits         14858     14860    +2

Impacted Files
Coverage Δ

...nternal/apiserver/route_get_token_account_pools.go
100.00% <ø> (ø)

internal/apiserver/route_get_token_accounts.go
100.00% <ø> (ø)

internal/database/sqlcommon/tokenbalance_sql.go
100.00% <100.00%> (ø)

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 f56d701...d894408. Read the comment docs.


Adding robots txt to rule gallery

Added robots text custom response to rule gallery

Hey @justngrok! Thanks for this one—loved hearing about it during the interlock last week. I think this is good to go, just trying to understand why the build is failing. I'm going to run the job again and keep my fingers crossed...

Yeah, first edit like this for me so not sure what I'm missing here.


16bit fixed point being returned as 8 bit?

qint8_t scvt_qs16_f32(float a, int fixed_point_position);
https://github.com/ARM-software/ComputeLibrary/blob/master/arm_compute/core/FixedPoint.h#L206

Thanks for reporting this. I can confirm that we already fixed that issue in our internal version.


Exclude compiler-generated code from coverage analysis

Current coverage is 87.16%

Merging #1546 into master will increase coverage by +9.88% as of 2150137

@@            master   #1546   diff @@
======================================
  Files          530     460      -70
  Stmts        32148   15024   -17124
  Branches      8794    1890    -6904
  Methods          0       0
======================================
- Hit          24844   13095   -11749
+ Partial       5135     706    -4429
+ Missed        2169    1223     -946

Review entire Coverage Diff as of 2150137

Powered by Codecov. Updated on successful CI builds.

:+1:

I'm not so sure this is what we want. The goal was to exclude the unreachable portions of the state machine generated for async methods, but it turned out to completely exclude all async methods as well as iterator blocks. I'm not sure what the right approach is yet.


FULL- VIDEOS] Kriti Sanon and Shaheer Sheikh's leaked video on Social Media

FULL- VIDEOS] Kriti Sanon and Shaheer Sheikh's leaked video on Social Media
Watch: Kriti Sanon, Shaheer Sheikh's Video From Do Patti Sets LEAKED
Watch 🟢 ➤ ➤ ➤ 🌐 Click Here To link (Full Viral Video Link)
🔴 ➤►DOWNLOAD👉👉 (Full Viral Video Link)
A leaked video from the sets of Kriti Sanon's upcoming film Do Patti has gone viral on social media platforms. The film also stars Kajol and Shaheer Sheikh. Besides playing the lead, Kriti has also produced the film with Kanika Dhillon.
The clip features Kriti and Shaheer standing in the middle of a busy street. While the actress is spotted in a powder blue skirt and sweatshirt, Shaheer is seen wearing jeans, t-shirt and an orange jacket.
A few other people are also seen around them in the video. However, the location of the shoot is not known yet. In the film, Shaheer plays Kriti's love interest.
Check out the video and photo here:
Read Also
Kriti Sanon Spotted Getting Cosy, Dancing With Boyfriend Kabir Bahia In Greece - Video Goes Viral
article-image
Kriti had earlier stated that working on Do Patti was a creatively satisfying experience.
 I am always creatively inclined towards things. I always wanted to be a part of films and tell stories in other ways besides just being an actor on set. I wanted to do this for a film for which I'm madly passionate about, in terms of its story, and Do Patti was that story. I was with Kanika through the film."


Tariffs: improve flexibility

Formulas
Currently, tariffs can be customized using tax and charges:
total := (price + charges) * (1 + tax)

where the base price  comes from an api, either for current price or as price forecast.
https://github.com/evcc-io/evcc/issues/16851 requested to add margin and uplifts to the calculation which was added in  https://github.com/evcc-io/evcc/pull/16855:
total := (price * (1 + margin) + charges) * (1 + tax) + uplifts

It has also been pointed out that this calculation is not flexible enough, as it does for example not take negative prices sufficiently into account.
We propose to revert https://github.com/evcc-io/evcc/pull/16855 to the original calculation and add an option for a custom, freely-definable calculation formula:
total := (math.Abs(price) + 0.02) * 1.19

Input to the formula is the api-provided price, output is the total price/cost.
Configuration of the formula could look like this:
grid:
  type: elering
  tax: 0.19
  charges: 0.02
  formula: (math.Abs(price) + charges) * (1 + tax)

Zones
Further requirement (https://github.com/evcc-io/evcc/issues/16829) is to add zoned calculation components ("tariffs"). If we add zones, similar to the fixed tariff, to the calculation, we need to determine which price components need be definable per zone. These additional components should be accessible from the formula in the same way:
total := (math.Abs(price) + charges) * 1.19

where price comes from the api and charges would be defined per zone.

So the following example would be possible?
formula: (price + math.Abs(price) * 0.33 + charges) * (1 + tax) + 0.22

Yes. But- as long as we do not add zone parameters- you would/could write this as:
(price + math.Abs(price) * 0.33 + 0.11) * 1.19 + 0.22

and only depend on price!

Nice :)

Yes, i think having this flexibility is great. Some vendors in Germany also started to offer dynamic tariffs with a fixed upper limit. Would be easy to implement concepts like this with this new formula.
Only drawback is, that writing a correct formula will be a challenge for non-technical users. Templates for specific tariffs would ease this.

Hi the zone question has been discussed before
here: https://github.com/evcc-io/evcc/discussions/12331#discussioncomment-8595277
There was even a commit (unfortunately I did not know how to test it so it was closed): https://github.com/evcc-io/evcc/pull/12871
Maybe it can be reused?

I just want to remind that feedin (here in .be) for dynamic contracts is dynamic too:
for instance
0.87 * price - 0.004
total := (price * (1 + margin) + charges)
see https://github.com/evcc-io/evcc/discussions/16465#discussioncomment-10839844


Is <IP_ADDRESS>7 usable?

Website says no, but remarkable-hacks now has support as does the rm2 framebuffer so not sure?

Not until the packages are updated and make it to stable.

Display update https://github.com/toltec-dev/toltec/pull/623
ddvk update https://github.com/toltec-dev/toltec/pull/627

I've installed toltec stable (as the bootstrap didn't use either package), and then built/installed both of these by hand. LGTM on my rm2 running <IP_ADDRESS>7

I've installed toltec stable (as the bootstrap didn't use either package), and then built/installed both of these by hand. LGTM on my rm2 running <IP_ADDRESS>7

I would recommend commenting on the pull requests themselves, and testing with the artifacts generated there.

@palfrey would you mind putting together a gist or something demonstrating how you got this working on the latest firmware? I’m really interested to try, but unsure when official support will come

@palfrey would you mind putting together a gist or something demonstrating how you got this working on the latest firmware? I’m really interested to try, but unsure when official support will come

I mostly just did the standard install and then you can copy individual .ipk's (either from local hand-built stuff or the artifacts in the PRs) and install those with opkg install <file>

@atze007 you can follow the normal installation procedure, but any framebuffer apps will require you to copy the updated installers for ddvk and display to your remarkable and run them manually.

We don't support running ddvk-hack's installer directly with toltec's package already installed. You'd need to grab the updated ipk artifacts from one of the open pull requests, or switch to the testing branch if the pull request has been merged.

I think https://github.com/toltec-dev/toltec/pull/635 solves this?


Update Calculator.java

Created the functions of add, subtract, multiply, and divide in the Calculator app.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


extension BetterAnime is the broken

Steps to reproduce
Attempt to invoke virtual method 'java. lang.String orgjsoup.nodes.Element. attr(java.lang. String)' on a null object reference
Expected behavior
Should show the animes that contains at Betteranime
Actual behavior
When I try to access Betteranime extension does not work
Crash logs
No response
Aniyomi version
<IP_ADDRESS>
Android version
Android 12.1
Device
Redmi Note 8
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

"If this is an issue with an extension, I should be opening an issue in the extension repository."

To add/fix extensions, make a github issue here if it does not exist already
https://github.com/aniyomiorg/aniyomi-extensions


Please default to P2P reference

I wanted to add a P2P reference. I don't normally use command pallet for that, but decided to give it a try.

Excellent! I did that and I was surprised it didn't "work". After a big of digging, I realized that this is a S2P reference. That sort of blew my mind. I was operating under the mental model of a project. The project was my object/noun. It's clear that devkit doesn't share this view and has the solution as the primary substrate for all operations.
I see now that DevKit offers P2P references in Solution Explorer (which I never use).

This seems to be something different. It seems to want to only want to add P2P references to projects within the same solution???

Note: I did this after removing the second project in the solution that is displayed in the earlier image.
I also see that DevKit offers S2P references in that same UI.

There is currently just one command pallet entry for "Add Project". So, its just a question of whether it adds the project reference to the solution or the project. The solution was chosen.
It seems like there are three options:

Status quo.
Expose command pallet commands for both S2P and P2P.
Have just one command pallet command (just like now) but switch it to P2P.

I like the last option best. I suspect many other users will prefer this. Please consider it.
At the same time, there are affordances given to special projects. Perhaps there is some bigger opportunity here.

It should also be noted that the CLI defaults to P2P.
I just did this:
$ dotnet add reference ../BacktrackerBenchmarks
Reference `..\BacktrackerBenchmarks\Benchmarks.csproj` added to the project.

This is in the same directory, which contains a project and a Devkit-generated solution file.
It's a yellow flag if the CLI and DK disagree on what the primary artifacts are at play. For the CLI, SLN operations are more off-to-the-side (which I'm good with).


Create a benchmark to track speed improvements / regressions

Considering a lot of time goes into optimizing the speed of UMAP it seems necessary to have some kind of bechmark/suite to evaluate performance.
EDIT:
A collection of useful links to get started:
Tracking System Capabilities: https://github.com/giampaolo/psutil
Datasets: https://www.tensorflow.org/datasets
Benchmarking: https://pypi.org/project/pyperf/

I would love to have that -- but I'm not sure of the best way to actually do that. I have very little experience/expertise in CI infrastructure, so yes, definitely help wanted.

First things first we would have to gather some datasets on which benchmarking makes sense.

I believe fetch_openml in sklearn can get us a bunch easily. At the least we can get MNIST and Fashion-MNIST with that. It would also be beneficial to have a word-embedding dataset on hand. For a sparse dataset I think a CountVectorized 20newsgroups is easy enough to get and should be somewhat representative. Ideally we would have a biology dataset as well, but I don't know where to get those.

I have code to easily use https://www.tensorflow.org/datasets/catalog
But that would require tensorflow as a dependency for benchmarks.

For a test suite to run in CI only for certain configurations (i.e. not by default) that's probably fine.

@lmcinnes Are you still looking to get this done? I've been working on a lightweight side-project (private repo at the moment, but I'm going to make it public soon!) that we could plug and play the UMAP calls into

I would love to have some benchmarks like that available, so yes, if you have the opportunity please let me know if there's anythign I can do from this end to help.

I would love to have some benchmarks like that available, so yes, if you have the opportunity please let me know if there's anythign I can do from this end to help.

Sounds good, will add that in. Feel free to assign this issue to me, and will get to it over the weekend/next week. I'm a big fan of this library, and I'm glad to help out. Do feel free to tag me on other issues that you may need a hand with :)

Sounds good, will add that in. Feel free to assign this issue to me, and will get to it over the weekend/next week. I'm a big fan of this library, and I'm glad to help out. Do feel free to tag me on other issues that you may need a hand with :)

I would love to have some benchmarks like that available, so yes, if you have the opportunity please let me know if there's anythign I can do from this end to help.

Sounds good, will work on the same. Please feel free to assign this issue to me, and will get to it over the weekend/next week. I'm a big fan of this library, and I'm glad to help out. Do feel free to tag me on other issues that you may need a hand with :)

I would love to have some benchmarks like that available, so yes, if you have the opportunity please let me know if there's anythign I can do from this end to help.

Sounds good, will work on the same. Please feel free to assign this issue to me, and will get to it over the weekend/next week. I'm a big fan of this library, and I'm glad to help out. Do feel free to tag me on other issues that you may need a hand with :)


Deploying my function app via VS Code failed

Repro steps:

Used CLI to init a function app
Used CLI to create a function  (demo Hello World app)
Used CLI to run locally and validated successful execution
Unable to get CLI to log in successfully so unable to publish
Created function app resources in Azure Portal, along with resource group, pricing / svc plan, associated with subscription, set up region / location etc.
Opened project in vs code & used Azure Functions: Create Function app in Azure and got the following error shown below.

I'm not clear what the suggestions below mean, can you help decypher what needs to happen to publish my function?
Action: azureFunctions.deploy
Error type: Error
Error Message: Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
Version: 0.18.1
OS: win32
Product: Visual Studio Code
Product Version: 1.38.1
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:361:81122
o extension.bundle.js:361:79501

I misspoke; I used vs code with the following command to try to publish my function to azure:
Azure Functions: Deploy to function app

That error means something is wonky in your VS Code configuration (the .vscode folder). You can copy the config here and I can try and figure out the issue. You can also try deleting that folder, deploy again, and it should prompt to re-add the config fresh

This is not closed -- the issue is NOT resolved.

Hi Eric,
In response to your note stating the following:
"If the browser isn't launched, you can try running az login --use-device-code to log in
If that still doesn't work, I recommend filing an issue directly on the func cli here: https://github.com/Azure/azure-functions-core-tools
For VS Code deploy, we'll use your other issue here: #1519"
The browser does launch, the problem is that the shell doesn't get successfully logged in, while the browser appears to successfully log in.
So Glenn recommended using the "cloud shell" -- Eric, do you recommend this also?  I have a project developed locally (just hello world), but the problem is in publishing the function app project to azure.  I've already successfully run it locally, but neither the command line nor the vs code app (which crashes near constantly) is able to publish this project to azure.
Glenn recommends using the cloud shell, Eric is talking about the .vscode directory in my project.  I'll start by publishing this .vscode directory.

Azure-Issue151.vscode.zip

I tried to publish the function from vs code again and got this error:
Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
I can't read any of the project files using vs code because it crashes every time I open a JSON or python file but I can edit them in the atom editor.
I don't see any reference to "azureFunctions.preDeployTask" anywhere, what file is that supposed to be in?
Also, I don't have a "tasks.json" file, should I have one?  What needs to go in there?  Can you point me towards docs on that?

I also tried Glenn's suggestion to use the "cloud shell" to deploy this.  I clicked the icon in the web console to open the cloud shell and got some boilerplate about project variables, including a resource group (different from the resource group I created when I created the function app project in the web console), a storage account, file share, etc.
I typed the "az" command to pull up the CLI (it responded with "Welcome to Azure CLI!" and a bunch of --help stuff)
Next I entered my az publish command and got the following error::
$ func azure functionapp publish mobile-header-fn-app --build remote
Unable to find project root. Expecting to find one of host.json in project root.
I do have a host.json in my project root (I'll upload it after this post), but I don't think my laptop and this web portal are in sync, or at least something is very likely misconfigured.
Just for good measure, I tried the same publish command from my cmd prompt.  First, I ran "az login", and that sprouted a browser tab, and I logged in via the browser without issue, but on the command line I see this error again:
az login
Note, we have launched a browser for you to login. For old experience with device code, use "az login --use-device-code"
Please ensure you have network connection. Error detail: HTTPSConnectionPool(host='login.microsoftonline.com', port=443): Max retries exceeded with url: /common/oauth2/token (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')],)",),))
I then tried again to publish with the same errors as always:

func azure functionapp publish mobile-header-fn-app --build remote
Unable to connect to Azure. Make sure you have the az CLI or Az.Accounts PowerShell module installed and logged in and try again

Eric, you said that you think that something is amiss in my .vscode project configuration.  Have you had a chance to look at the zipped archive of that I uploaded?

host.zip

Above is the uploaded host.json in a zip archive... thanks.

I don't see any reference to "azureFunctions.preDeployTask" anywhere, what file is that supposed to be in?

.vscode/settings.json

Also, I don't have a "tasks.json" file, should I have one? What needs to go in there? Can you point me towards docs on that?

Yes you should have one in your .vscode folder. https://github.com/microsoft/vscode-azurefunctions/wiki/Project-Structure

Eric, you said that you think that something is amiss in my .vscode project configuration. Have you had a chance to look at the zipped archive of that I uploaded?

Having problems with the zip. It should be at most 4 smallish json files - can you just copy the contents into a comment here? Also did you try this as mentioned above: "You can also try deleting that folder, deploy again, and it should prompt to re-add the config fresh"

Absolutely, here you go:
extensions.json:
{
"recommendations": [
"ms-azuretools.vscode-azurefunctions",
"ms-python.python"
]
}
launch.json:
{
"version": "0.2.0",
"configurations": [
{
"name": "Attach to Python Functions",
"type": "python",
"request": "attach",
"port": 9091,
"preLaunchTask": "func: host start"
}
]
}
settings.json:
{
"azureFunctions.deploySubpath": "mobile-header-fn-app.zip",
"azureFunctions.projectLanguage": "Python",
"azureFunctions.projectRuntime": "~2",
"debug.internalConsoleOptions": "neverOpen",
"azureFunctions.preDeployTask": "func: pack"
}
tasks.json:
{
"version": "2.0.0",
"tasks": [
{
"type": "func",
"command": "host start",
"problemMatcher": "$func-watch",
"isBackground": true
}
]
}

Well... that's slightly more readable.  Please let me know what you find.

So does that error mean that I should delete this property from settings.json?
"azureFunctions.preDeployTask": "func: pack"
that error being this:
Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.

Here are the newly created .vscode files -- it looks like tasks.json has grown a bit
extensions.json:
{   "recommendations": [     "ms-azuretools.vscode-azurefunctions",     "ms-python.python"   ] }
launch.json:
{     "version": "0.2.0",     "configurations": [         {             "name": "Attach to Python Functions",             "type": "python",             "request": "attach",             "port": 9091,             "preLaunchTask": "func: host start"         }     ] }
settings.json:
{     "azureFunctions.deploySubpath": "mobile-header-fn-app.zip",     "azureFunctions.pythonVenv": ".venv",     "azureFunctions.projectLanguage": "Python",     "azureFunctions.projectRuntime": "~2",     "debug.internalConsoleOptions": "neverOpen",     "azureFunctions.preDeployTask": "func: pack" }
tasks.json:
{     "version": "2.0.0",     "tasks": [         {             "type": "func",             "command": "host start",             "problemMatcher": "$func-watch",             "isBackground": true,             "dependsOn": "pipInstall"         },         {             "label": "pipInstall",             "type": "shell",             "osx": {                 "command": "${config:azureFunctions.pythonVenv}/bin/python -m pip install -r requirements.txt"             },             "windows": {                 "command": "${config:azureFunctions.pythonVenv}\\Scripts\\python -m pip install -r requirements.txt"             },             "linux": {                 "command": "${config:azureFunctions.pythonVenv}/bin/python -m pip install -r requirements.txt"             },             "problemMatcher": []         }     ] }

I tried to run the deploy from vs code again.  Note that I have only the Azure Functions extension added to vs code, which pulls Azure Accounts extension with it.  I tried to run the deploy using the "Azure Functions: Deploy..." command and it notifies of errors again:
Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
There was another notification that said something like See errors from this deployment but unfortunately when I clicked it to see the errors, vs code crashed again.

I uninstalled, fully cleaned my env and reinstalled.  Then installed only the azure functions extension and ran the "Azure Functions: Deploy to Function App" command.
I'll now give a play by play of the next several things that happen

I click "Deploy" and I quickly see these:

Note I am logged in to azure:

I can't click the lower of the two dialogs about task errors because vs code will crash.  How do I check out the errors it's reporting?

I closed the lolwer box since clickingto open will crash, but I clicked the top box to "report an issue" and it opened an issue in github with the following text prep-populated.  It doesn't make a huge amount of sense to me, but maybe it will mean more to you guys?  Here it is:
`
Repro steps:

Action: azureFunctions.deploy
Error type: Error
Error Message: Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
Version: 0.18.1
OS: win32
Product: Visual Studio Code
Product Version: 1.38.1
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:361:81122
o extension.bundle.js:361:79501

`

Hey Eric, you mentioned the following:
If the browser isn't launched, you can try running az login --use-device-code to log in If that still doesn't work, I recommend filing an issue directly on the func cli here: https://github.com/Azure/azure-functions-core-tools For VS Code deploy, we'll use your other issue here: #1519
So, the browser is launched, but it doesn't log the cmd prompt in even though the browser logs in without issue.  It seems like the link from cmd prompt to browser is good but the back link from the browser to the cmd prompt isn't working, do you know how to debug this?

WRT this error:
Error Message: Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
Should I remove this setting from settings.json?:
"azureFunctions.preDeployTask": "func: pack"
What is that command doing?  Is it relevant for python?  Or I could add that task to tasks.json?  What would that look like?

Just for completeness here's the zip file I created manually, perhaps something is not correct with it?  I have been browsing through it but I'm just starting out with this stuff so I'm not seeing anything obviously wrong
mobile-header-fn-app.zip

Next, instead of removing the preDeployTaks from setitngs.json, I am trying to add the preDeployTask from settings.json to tasks.json as the last item in the first object in the list -- here's the contents of the file now:
{     "version": "2.0.0",     "tasks": [         {             "type": "func",             "command": "host start",             "problemMatcher": "$func-watch",             "isBackground": true,             "dependsOn": "pipInstall",             "azureFunctions.preDeployTask": "func: pack"         },         {             "label": "pipInstall",             "type": "shell",             "osx": {                 "command": "${config:azureFunctions.pythonVenv}/bin/python -m pip install -r requirements.txt"             },             "windows": {                 "command": "${config:azureFunctions.pythonVenv}\\Scripts\\python -m pip install -r requirements.txt"             },             "linux": {                 "command": "${config:azureFunctions.pythonVenv}/bin/python -m pip install -r requirements.txt"             },             "problemMatcher": []         }     ] }
Then I try to deploy again...

But I just get back to the original error:
Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
Here's what the dialog looks like, it's specifically a notification from the Azure Functions extension it would appear:

I hit the "Report an issue" button shown above and got this:
`
Repro steps:

Action: azureFunctions.deploy
Error type: Error
Error Message: Failed to find preDeploy task "func: pack". Modify the setting "azureFunctions.preDeployTask" or add that task to tasks.json.
Version: 0.18.1
OS: win32
Product: Visual Studio Code
Product Version: 1.38.1
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:361:81122
o extension.bundle.js:361:79501

`

The "func: pack" task is what is creating the zip file. If you don't want that task (since it's causing so many issues) you can try this preview feature: https://github.com/microsoft/vscode-azurefunctions/wiki/Server-Side-Build

I also mapped my directory to a new drive letter, so now have this:

Any particular reason you're doing this? I'm guessing you might have better luck if you use the local file system

I moved my entire project to the C:\ drive -- but no change in behavior whatsoever.
I also tried the instructions in this:
https://github.com/microsoft/vscode-azurefunctions/wiki/Server-Side-Build
However, that didn't work either.

Closing due to lack of activity and because this thread covered many disparate issues that were hard to reproduce on our end. Hopefully you got it working or found a better solution that works for you


Vulkan geometry instance assigned the wrong shader instance uniform when rendering

Godot version
4.0.dev (86460db231)
System information
W10, Vulkan Clustered, RTX2060
Issue description
When assigning a number of instance uniform values to a set of mesh instances, some of the meshes are using the wrong instance uniform.
This grid of boxes should be forming a smooth red/green gradient:
(10x10 grid)

It seems to be misaligned in counts of 64
(16x16 grid)

Steps to reproduce
Create an assortment of geometry instances, and assign a unique instance uniform value to each.
Notice how some use the wrong values when rendering.
Minimal reproduction project
InstanceUniformBrokenReprod.zip

I can confirm this on commit c89061e98 (Linux, GeForce GTX 1080).
This also occurs after switching to the Vulkan Mobile backend in the project setting, so this is not due to clustering:

Can confirm this in 28174d531b7128f0281fc2b88da2f4962fd3513e
Got there the long way... as I did not realize that this was just about the number of instances having to exceed 64 (seems like).
In my repro I also found that timing had an effect.


Cycle 4: Specreduce and other spectrocopy work (Pickering)

Proposal to continue on with specreduce and other astropy-related spectroscopy efforts.

Please remove travel from this request.  Since SPIE is coming up soon, I would recommend opening an issue (separate from Cycle 4) for that request.

Please remove travel from this request. Since SPIE is coming up soon, I would recommend opening an issue (separate from Cycle 4) for that request.

done and done.

While obviously we'd love to fund this for the full amount listed, if we need to squeeze the budget is there a minimum number of hours that would make sense?

Please react to this comment to vote on this proposal (👍, 👎, or no reaction for +0)

While obviously we'd love to fund this for the full amount listed, if we need to squeeze the budget is there a minimum number of hours that would make sense?

i was traveling so didn't get a chance to follow up on this. i would say 1 hr/week would be kind of the bare minimum to cover maintainer duties with no extra development time. i can amend my proposal to say this if there's time and it would be useful.

Hi Tim,
I'm writing on behalf of Astropy's Finance Committee regarding the outcome of your funding request. I will shortly open a new issue to track the progress.
We are pleased to be able to let you know that, following consultation with the community, we are able to approve this request. We can currently fund the Year 1 amount of $5,760 (US) to carry out the project. Funding beyond this amount will be contingent upon the availability of funds. (We will be using the full budgets of all of the approved requests to craft future grant and funding proposals.) We assume you will be using this full year 1 budget, if that is not the case please contact us immediately.
Ana Gabela and I will be your contacts on the Finance Committee to facilitate this award. Please get in touch with us if you have any questions or concerns. Please do not reach out to NumFOCUS directly.
In addition, new to this funding cycle, is the assignment of a COTR (Contracting Officer's Technical Representative) to each funded project. This concept is borrowed from government funding agencies, although it is to be stressed that Astropy's goal is to make the COTR role as low-overhead as possible. The COTR’s primary responsibility is to make sure the work is happening at the expected pace and, if necessary, to be a liaison between the funded project and the Finance Committee or CoCo. The COTR for your project will be assigned shortly and we’ll also be sending out more details about how we see this working.
The next steps are:

Please make sure that you are registered with Open Collective: this is the system you will use to submit your invoices for payment. Note that your invoices should be submitted through the NASA project on Open Collective.
We will reach out through NumFOCUS, Astropy's Fiscal Sponsor, to set up an “independent contractor agreement” (ICA; effectively a contract) and formal scope of work for the project.

Congratulations --- we are really looking forward to seeing you put these funds to good use!
Kelle, on behalf of the Astropy Finance Committee


Error importing handler: taxii_services.PollRequestHandler

Getting the following response when attempting to poll using the latest version of Yeti (not sure if this belongs here or there):
Traceback:
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
112.                     response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/django/views/decorators/csrf.py" in wrapped_view
57.         return view_func(*args, **kwargs)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/taxii_services-0.3-py2.7.egg/taxii_services/views.py" in service_router
85.         handler_class = getattr(module, class_name)
Exception Type: AttributeError at /services/poll/
Exception Value: (u'Error importing handler: taxii_services.PollRequestHandler', <type 'exceptions.AttributeError'>, AttributeError("'module' object has no attribute 'PollRequestHandler'",))
Sorry for the encoded stuffs, this is coming back as a Django error in the taxii_msg I get back. Latest master pull of Yeti.

@mgoffin,
Thank you for pointing this out. A recent commit by @gtback broke this (https://github.com/TAXIIProject/django-taxii-services/commit/d412c5af39efb7b71268b2a6add288f5b8f37784#diff-1db46ba9eb18f3e24afc09d5ef9247ea). I'll assign this issue to him for remediation.
-Mark

The commit I made was deliberate. I'm trying to get away from having import X import the entire library (for X in cybox, stix, maec, libtaxii, ...), which sometimes takes a few seconds and is inefficient for scripts called in a loop.
@mgoffin: You should be able to change to the following: from taxii_services.handlers.default import PollRequestHandler
@MarkDavidson If there's consensus to do it the other way, I can revert that commit.

@gtback That change would be in TAXII land, not the code I am writing. I can fix it locally but it would probably make more sense to fix it in taxii_services, yes?

@gtback
The use case is to be able to configure the model (from an admin page, not from code) in a succinct manner. Basically, the way it WAS organized, you'd have some form that's akin to:
Handler Name = MyCoolHandler
Handler = taxii_services.PollRequestHandler

If the path is longer, the form looks like:
Handler Name = MyCoolHandler
Handler = taxii_services.handlers.default.PollRequestHandler

My goal was to keep the Handler = ... part as small as possible. I don't know if I did that the best way (or if there's a way to keep the import lines working while also not making the import take too long).
So I think we're after two separate use cases. Mine is more of a user experience use case, I think, while yours is a code organization / code execution use case.
Thoughts?
Thank you.
-Mark

@mgoffin: Sorry, I checked to see if it might be something in YETI, but didn't see "handler": "taxii_services.PollRequestHandler" in the initial_data.json, hence my (incorrect) assumption that it was in something you wrote.  My preference would be to fix it in YETI.
@MarkDavidson: I've never liked exposing Python import paths to users, particularly in a web-based admin console (it's tolerable in a text config file, IMO). I understand why keeping them shorter is preferable, but if we decide to stick with that, I think we should catch the attribute error and raise a more sensible error (i.e. "Could not load Handler class taxii_services.PollRequestHandler for MyCoolHandler. Please check the Python path.")

@mgoffin : I released django-taxii-services 0.4 on PyPI, and updated YETI to include the new version in requirements.txt. Can you fetch the latest version of both projects and verify that this problem has been resolved?

Hoping I have some time today to test but if I don't I'll try to get to it as soon as possible!


Remove unnecessary src parameter to Stream

Codecov Report

Merging #68 into master will decrease coverage by 0.05%.
The diff coverage is 100%.

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


Roll engine ff1bcdc00941..7ee3a31c55d4 (1 commits)

https://github.com/flutter/engine/compare/ff1bcdc00941...7ee3a31c55d4
git log ff1bcdc00941188ba27123a4857b443e3c9392bb..7ee3a31c55d45d70261a22f9d1e280e71f4f03d3 --no-merges --oneline
7ee3a31c5 Roll src/third_party/dart 907c514c89..86b5304ccd (27 commits)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6610538858545152

Commit queue failed; closing this roll.


Rename Windows Metrics(swap -> virtual memory) & fix ui(config) Memory RAM (MB) is blank.

Fix #12105

[ ] Add a unit test to verify that the fix works.
[x] Explain briefly why the bug exists and how to fix it.
[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #12105.
[ ] Update the CHANGES log.

With your metric name changes, the e2e tests may need to change, or CI will fail.

https://github.com/apache/skywalking/tree/master/test/e2e-v2/cases/win
Here is the e2e, could you verify the e2e as well.
And follow the PR template to update changes.md.


Build riscv64 problem

build methods:
sh scripts/build-sparrow.sh riscv64

build's result hints this:
sparrow/kernel/include/arch/riscv/arch/kernel/machine_timer.h:18:10: fatal error: opentitan/timer.h: No such file or directory
   18 | #include "opentitan/timer.h"
      |          ^~~~~~~~~~~~~~~~~~~
compilation terminated.

where can i found this timer.h or how to create it?
Thanks a lot.

On Sun, Oct 23, 2022 at 6:55 PM YaSuoInTheHouse @.***>
wrote:

build methods:
sh scripts/build-sparrow.sh riscv64
build's result hints this:
sparrow/kernel/include/arch/riscv/arch/kernel/machine_timer.h:18:10: fatal error: opentitan/timer.h: No such file or directory
18 | #include "opentitan/timer.h"
|          ^~~~~~~~~~~~~~~~~~~
compilation terminated.
where can i found this timer.h or how to create it?
opentitan/timer.h comes from the opentitan
https://github.com/lowRISC/opentitan project but it's  not what you want.
Sparrow on riscv should use the upstream (seL4) support for any non-sparrow
platform but unfortunately the riscv support was removed when brought up
the system on our (riscv32) platform. To make this work on whatever riscv
system you wanted you'll need to merge from upstream (which will eliminate
the dep on opentitan).

-Sam

Message ID: @.***>


Evaluate authenticator in the controller context

Hi, thank you for such an awesome and complete library!
In my use case, I am using application_controller_mixin with token_authenticator that referencing a method inside the module defined in application_controller_mixin where, similar with https://github.com/RIPAGlobal/scimitar/issues/57, the current_company is the tenant and we get it via subdomain
Scimitar.engine_configuration = Scimitar::EngineConfiguration.new({
  application_controller_mixin: CurrentCompanyMixin,

  token_authenticator: Proc.new do | token, options |
    api_key = current_company.token

    ActiveSupport::SecurityUtils.secure_compare(token, token)
  end
})

However, the current implementation of passing the Proc directly to authenticate_with_http_token is executed not in the context of the Scimitar::ApplicationController, so token_authenticator doesn't have access to any controller's method
In the Rails example of authenticate_with_http_token https://api.rubyonrails.org/classes/ActionController/HttpAuthentication/Token.html. authenticate_with_http_token can have access to it's outer block
class ApplicationController < ActionController::Base
  before_action :set_account, :authenticate

  private
    def set_account
      @account = Account.find_by(url_name: request.subdomains.first)
    end

    def authenticate
      case request.format
      when Mime[:xml], Mime[:atom]
        if user = authenticate_with_http_token { |t, o| @account.users.authenticate(t, o) }
          @current_user = user
        else
          request_http_token_authentication
        end
      else
        if session_authenticated?
          @current_user = @account.users.find(session[:authenticated][:user_id])
        else
          redirect_to(login_url) and return false
        end
      end
    end
end

This PR changes how Scimitar evaluates token_authenticator and basic_authenticator by using instance_exec, so they can have access to any method defined in the controller.

This'll go into v2.7.0 - many thanks for the contribution 😄

For general coding style & future reference in other PRs

Thanks! And sorry, our default style guide is different so I did it out of my own habit. I will try to remember next time.
What do you think about adding rubocop config to the repo to enforce the style guide?

@tejanium

What do you think about adding rubocop config to the repo to enforce the style guide?

It's not that big a deal in general and usually people tend to follow the "keep the code in the same style as the code around it" rule. I've not had much success with RuboCop - it takes an inordinate amount of configuration and always seems to find something  to complain about.


1365 bek / Etienne Eid / https://github.com/etienneeid

https://github.com/orgs/projets2017cl/teams/nfp103/members


(BKR-1499) Adds Beaker::Host#chown, #chgrp, and #ls_ld

Adds chown and chgrp methods so that hosts can handle per-host peculiarities, e.g. Cygwin hosts must call cygpath -u to get UNIX/POSIX style paths to pass to chown, because Cygwin chown does not support Windows-style paths.
This helps with overall efforts to make host abstractions more platform-independent.
Needs PSWindows coverage.

Failing on
15:16:11 Minitest::Assertion: Should have found testuser in `ls -l` output.
15:16:11 Expected /testuser/ to match "-rw-r--r-- 1 Administrator None 0 Jul 12 22:16 C:/cygwin64/tmp/beaker.iXB2Oe\n".

This is because Cygwin doesn't properly support Windows-style paths in all commands. Chown doesn't work, and ls seems to half work:
$ cygpath -m $(mktemp -td #{beaker}.XXXXXX)
C:/cygwin64/tmp/beaker.iXB2Oe
$ chown testuserblah C:/cygwin64/tmp/beaker.iXB2Oe
# exit 0
$ ls -ld C:/cygwin64/tmp/beaker.iXB2Oe
drwxr-xr-x 1 Administrator None 0 Jul 12 23:23 C:/cygwin64/tmp/wintest.bW9TYp
$ ls -ld /tmp/wintest.bW9TYp
drwx------+ 1 Administrator None 0 Jul 12 23:23 /tmp/wintest.bW9TYp
$ chown testuserblah /tmp/beaker.iXB2Oe
# exit 0
$ ls -ld C:/cygwin64/tmp/beaker.iXB2Oe # lots of WTF
drwxr-xr-x 1 Administrator None 0 Jul 12 23:23 C:/cygwin64/tmp/wintest.bW9TYp
$ ls -ld /tmp/beaker.iXB2Oe
drwx------+ 1 testuserblah None 0 Jul 12 23:23 /tmp/wintest.bW9TYp

I'm going to fix this for now by implementing an ls_ld method to call ls -ld on the system, which Windows can wrap with a cygpath call just like chown and chgrp. However I consider this strong evidence against Cygwin hosts outputting anything but POSIX style paths.

Only failing on a single smoke test host, which timed out. Assuming transient.
Note: this needs to merge before #1531.

Jenkins, retest this please.


TP-702 Edit certificate descriptions

Summary
Allow the user to edit certificate descriptions
Screenshots
List page

Edit page

Confirmation page

Tariff changes page

Codecov Report

Merging #286 (10ac2d4) into master (e73088a) will decrease coverage by 0.13%.
The diff coverage is 68.42%.

@@            Coverage Diff             @@
##           master     #286      +/-   ##
==========================================
- Coverage   89.11%   88.97%   -0.14%
==========================================
  Files         251      251
  Lines       12532    12564      +32
  Branches      947      951       +4
==========================================
+ Hits        11168    11179      +11
- Misses       1182     1200      +18
- Partials      182      185       +3

Impacted Files
Coverage Δ

certificates/models.py
84.12% <35.71%> (-13.88%)
:arrow_down:

certificates/views.py
82.08% <67.85%> (-11.79%)
:arrow_down:

additional_codes/import_parsers.py
100.00% <100.00%> (ø)

additional_codes/models.py
90.62% <100.00%> (-0.15%)
:arrow_down:

additional_codes/serializers.py
100.00% <100.00%> (ø)

certificates/forms.py
68.85% <100.00%> (+0.51%)
:arrow_up:

certificates/urls.py
100.00% <100.00%> (ø)

common/tests/factories.py
99.64% <100.00%> (ø)

common/urls.py
100.00% <100.00%> (ø)

footnotes/import_parsers.py
100.00% <100.00%> (ø)

... 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 e73088a...10ac2d4.


Switch the build to JDK 17

This switches the build to use JDK 17 with --release 8 (or whatever is appropriate for the module).
JDK 8 is still required for:

running tests (it is regulated by the surefire.jdkToolchain property),
providing jconsole.jar for log4j-jmx-gui.

To compile log4j-jmx-gui running JDK 17 a JAVA_HOME_8_X64 environment variable must be provided and contain the location of the JDK 8 installation

@vy,
I updated the PR so that you can run the build with whatever JDK newer or equal to 11, as long as:

you set the JAVA_HOME_8_X64 environment variable for JConsole,
you have a valid toolchains configuration with at least a JRE 11+ (for log4j-jpl).

The first condition will drop when we get rid of log4j-jmx-gui, the second one when Maven will generate an implicit JDK toolchain configuration (I have a PR somewhere).

@ppkarwasz, thanks so much for the quick response! I have some questions/remarks:
Toolchains
It feels odd to me to require both toolchains (which is the Maven-native way of providing multiple JDKs) and an environment variable for JDK 8. I understand that you intend to move log4j-jmx-gui to its own repository and then we won't need the JAVA_HOME_8_X64 environment variable anymore. Though the need for toolchains will still be there. I think, if possible, we should pick one of the two plans:

Embrace 17 & 8 in toolchains (no JAVA_HOME_8_X64 environment variable!) in this PR, later on remove log4j-jmx-gui, and consequently remove the need for toolchains and totally switch to 17
Remove log4j-jmx-gui (this can be done in this PR!) and only rely on 17 (no toolchains!)

I am in favor of the 2nd approach here. But maybe I am missing something obvious.
>=11 vs 17
I also would prefer a fixed JDK version (that is, as the name of this PR suggest, 17) rather than leaving it open. This will help us to align the build and development experience everywhere; in CI, in personal development machines, etc. The way it feels now is like a ticking bomb for "build doesn't work on my machine while using Java X" posts.

The need for toolchains comes from a couple of JDK 8-specific tests in log4j-api and log4j-core: StackLocator and a couple of other classes has an implementation that only works on JDK 8 (or JDK <11).
If you run those tests on JDK 11, they might succeed, but they will not be testing the code they are supposed to test.
I don't see a problem in running the build with JDK 11 (unless it is for a release): if there are differences in building experience between JDK 11 and JDK 17, we should know about it.

@vy,
I refactored this PR by replacing the dependency on jconsole.jar with two dummy classes, that are excluded from packaging afterwards. The new requirements for the build are:

the build should run on JDK 11 or later without a toolchains configuration or an environment variable; it skips a couple of JRE 8 specific tests,
there is a profile java8-tests that runs tests using a JRE 8 toolchain. This profile is used by the CI and requires two toolchains configurations: a JRE 8 and JRE 11 or higher (for log4j-api-java9, log4j-core-java9 and log4j-jpl).

For simplicity I also replace the maven-bundle-plugin with maven-jar-plugin in log4j-jmx-gui. This artifact does not need to be an OSGI bundle.

In the end JDK 17 was too ambitious: many tests still fail there.
In its final form this PR:

switches the build to JDK 9+ without requiring toolchains or anything else. On JDK 11 all tests pass, on JDK 17 there are some test failures (mainly due to the indiscriminate use of reflection).
the java8-tests profile (used by the CI) requires a toolchains.xml configuration with a JDK 8 toolchain.


Support specifying a gap between highlight and tooltip

I use tooltips without arrows, so I like to display a small gap between the highlight and the tooltip.  I currently realize this gap via padding around the tooltip composable, in all four directions.  But this approach will be wasteful in cases where the tooltip is sized not to run off the left or right side of the screen.  I could switch on position to put the padding only on the side that needs it (adjacent to the highlight), but a centralized solution might be better, in case other users need the same thing.
The gap could be specified on UnifyCoachmark, and defaulted to 0.dp for backward compatibility.

I can do this on my fork and then create a PR, if you give me the green light.  It'll probably be a couple days before I start on it.

sure will do this

Hey @glasheen99 , I checked this
Since Tooltip is configurable client side so let client do it
adding a extra param would not be a big change but with library evolving params will keep on increasing and hence it may create confustion for end users
Since user knows in which direction we are putting tooltip, so they can adjust padding direction accordingly
What do you say ?

It would have been useful for me, and perhaps other users eventually (to save having to "reinvent the wheel" on tooltips without arrows), but I'm trying a different approach now.


Update Elemental quickstart screenshots

Using:

Rancher 2.8.3
Elemental 1.4

Add missing UI extension screenshots.
Follow up from: https://github.com/suse-edge/suse-edge.github.io/pull/210

@atanasdinov can you please provide the UI extension screenshots for Rancher and Elemental?

I don't have such - the initial skeleton of the page was supposed to include them so I created this as a follow up.

@atanasdinov Do you want me to close this issue then?


Added Indonesian translation

Hello @samuelmeuli
I really like your application, and I have added new translations into my country's language, Indonesian. But it is still incomplete, and I will try to complete it. Thank you, hopefully you can help.

Awesome, thanks for your contribution! What can I help you with?

There is nothing you need to help, just continue to develop this application :)
I will improve the Indonesian translation in the future with a more perfect vocabulary

Sorry, I see in the contribution section , it asks to test out how to change the app language and says "(see steps below)" but doesn't not have steps below. How is it that you verified this?

This should now read "see steps above" (in the "Development" section).

@samuelmeuli my apologies, I am sorry, but I still don't see a way to change the language display when it is run? I'm feeling blind on this 😅My app loads in english no matter what and not the new translation file I added in. I have gone to the developer console to try and change it and the preferences, but sadly I can't find a way.

@dmcfaddengalway My bad, I should've explained more clearly. The app will use the system language, there's no option to set it for the app only. So if you set the language in your OS settings (you might also have to reboot), add the translation files and run the app from your Git clone, it should use the correct language :)


Allow error-handling plugin to accept transform function [fusion-plugin-error-handling] feature(UBERCOM-2530)

resolves https://github.com/fusionjs/fusionjs/issues/698

Hi! Thank for the PR. I've got some feedback.
I'm not sure serializing a function and adding all this logic is necessary for what should be a
fairly straightforward error handler. I think it'd be better to apply any logic you need to the /_errors endpoint by overriding or enhancing the ErrorHandlerToken. FWIW, all the information you mentioned in the linked issue should be readily available on the koa context, ctx.request.header.referer, ctx.request.header.cookie, etc.

I've tried implementing it purely using the ErrorHandlerToken (by modifying the @uber/fusion-plugin-error-handling package) but unfortunately, the ctx available to the error handler is the ctx for the request to /_errors, and no information available from the original request besides what gets posted by the client in the onerror function. The problem is that any errors coming from the frontend is another request all together, so we don't have access to any information about the request that is triggering errors in the first place, unless we do something like cache request data with a key so we can find information about the original request. I agree that serializing a function is kinda ugly though - especially since it wont be going through the proper webpacking channels (the function must be self-contained with no imports).
Can you advise me on an implementation path?

but unfortunately, the ctx available to the error handler is the ctx for the request to /_errors...

This is correct, but the referer of the request is the page from which the /_errors request is being made. For example, the ctx.request.header.referer for a /_errors request made from the www.mysite.com/some/path is www.mysite.com/some/path. In addition, the cookies are sent with this request, so you can access all that.

Can you advise me on an implementation path?

import {createPlugin} from 'fusion-core';
import ErrorHandlingPlugin, {
  ErrorHandlerToken,
} from 'fusion-plugin-error-handling';

export default function (app) {

  // Mock some browser error
  if (__BROWSER__) {
    setTimeout(() => {
      throw new Error('Log this error!');
    }, 1000);
  }

  if (__NODE__) {
    app.register(ErrorHandlerToken, MyErrorHandlingPlugin);
    app.enhance(ErrorHandlerToken, baseHandler => {
      // [How to enhance](https://fusionjs.com/api/fusion-core#enhancing-a-dependency)
      return createPlugin({
        provides() {
          return function (body, captureType, ctx) {
            // Do whatever you want here
            console.log(ctx.request.header.referer);
            console.log(ctx.request.header.cookie);

            // But still pass off to the plugin you are enhancing
            return baseHandler(body, something, ctx);
          };
        }
      });
    });
}

@micburks this works for what we need, thanks!


i18n tooling + translations

Note: @Oppia don't have the time to review the upstream pull request (https://github.com/oppia/xblock/pull/2), hence they can't get it merged. Opening it here as an intermediate solution.

We'd like this XBlock to be translated to Japanese and other languages, I've also supported JavaScript translations via
django-statici18n.
Before you continue, take a look at edx/i18n-tools and edx/cookiecutter-django-app.
Summary of the Changes

Added a Makefile to help manage the translations. The file is copied and modified from: edx/cookiecutter-django-app
Created a locale link to oppia/translations to allow the i18n_tools find the po/mo files
Added oppia/translations/config.yaml which works as the configuration file of the i18n_tools command
Added requirements-dev.txt to pin development packages. Used for now only for translations.

TODO

[x] Support JavaScript translations as textjs.po
[x] Document how to update the strings properly from Transifex
[x] Manually test that the the dummy Esperanto translation is working

Cheese
Studio

LMS

@melvinsoft The @Oppia team don't want to merge this PR. Could you please take a look?


Status field not being updated for the attachment resource

I have noticed when I use a DecoratorController, and try to send the response with status for an attachment resource, although it gets reflected in the attachment resource for the very first time, but for the subsequent updates in the status field of this attachment resource, metacontroller just ignores doing any changes - its even missing in the mergepatch calculated.
sample logs from metacontroller:
in request i see this: (expected)
"status":{"conditions":[{"lastTransitionTime":"2023-03-06T10:23:53Z","status":"True","type":"CUD"}]}}}},"related":{},"finalizing":false}}

in response i see this: (expected)
"status":{"conditions":[{"type":"CUD","status":"True","lastTransitionTime":"2023-03-09T07:31:52Z"}]}}]}}

Notice the time change in the request and response.
Now, in this log, metacontroller gives the expected change in spec in attachment cr, but I cant see the status bit here
{"level":"Level(-5)","ts":1678347112.267575,"msg":"Diff between observed and desired","mergePatch":{"spec":{"data":"{\"maxclients\":\"9800\",\"repl-ping-replica-period\":\"10\"},,\"runAsNonRoot\":true,\"runAsUser\":7777777}}"}}}

hi @Shashankft9 , I saw https://github.com/metacontroller/metacontroller/pull/776, however this is not the way how I would like to handle it, as it is arbitrary changing the behawior. In some cases  people do not controll the races in resources.
I would see that it will be controllable behaviour per child/attachement, so change in the config API, I still need to wrap an idea regarding this, as have some other config related changes in API types (i.e. emitting events)


Bugfix/fix base

I fixed the base param to work with a plain '/' as the base, it did not work at all before.
I also removed the history/hash push state on initial load... this should probably be reworked more later, but for me it was causing a bug where I would have an extra duplicate history state in my browser history after first hitting the app using the router. May need to only push the state on initial load if the base isn't already in the window path, or we are using hash and the hash isn't already in the window path. Presumably in most use cases though, the user will be entering at a valid path because the server will 404 other paths, so I didn't change it yet.
Another change here is: I renamed "notStrictRouting" to "strictRouting" for readability. I also made it false by default, because the way we are doing "redirects" is somewhat flawed and it always causes extra duplicate history states to get pushed to the browser history, making the back button not behave exactly as the user might expect. So I think the recommendation should be to leave strictRouting false, and make your SPA render a 404 page (or something like that) if the new route object was empty. I.e. prefer to handle these situations on a per-app basis instead of in the router.
Hope this all makes sense. I am going to merge this in my fork, so I hope ya'll agree with the changes. I know this is a lot of code changes for a single "bugfix" but I feel like these are some fundamental issues that can't really be tackled separately

This looks great, nice work

@747823 can you squash your commits and then I'll merge?


[backport | stable-1.11] virtiofs fixes

The commits backported are bringing fixes for:

#2686 - katatestutils should use config.virtioFSDaemon
#2682 - Runtime crash when virtiofsd is not executable

@chavafg @GabyCT Seeing this in the travis logs:
Check versions in branch master
~/gopath/src/github.com/kata-containers/packaging ~/gopath/src/github.com/kata-containers/runtime
INFO: Tagging Kata Containers with version 1.11.1
INFO: Check all repos has version 1.11.1 in VERSION file
INFO: agent is in 1.11.0-rc0
ERROR: agent is not in version 1.11.1
The command ".ci/versions_checker.sh" failed and exited with 1 during .

Looks like the version being checked in the agent repo is coming from master branch instead of 1.11.1 branch. Can you take a look?

@jcvenegas any idea? ^

@fidencio @chavafg could you try to restart travis, probably was in the middle of the release process ?

I've rebased atop of origin/main and then force-pushed to my branch in order to re-trigger the travis-ci

/test-ubuntu

/test-clh

/test

my bad, I did not refresh my browser tab


Investigate behaviour of comparisons over different types

What happens if a signed type is compared to an unsigned type? Is this even possible?

Closing as we do not plan on supporting unsigned integers.
It seems annoying to implement unsigned integers without create a lot of code bloat at the moment as there is an entirely separate set of LLVM intrinsic functions that need to be supported for unsigned integers arithmetic.
Moreover, it appears like a lot of tedious (although not very interesting code) would need to be written to determine whether to constrain an integer input as signed or unsigned (since the LLVM only specifies the bit-width and not whether the data is signed or unsigned—signed or unsigned behaviour simply emerges from the different set of LLVM intrinsic functions called for arithmetic).


Initial Slide progresses twice on rerender (...sometimes?)

See https://codesandbox.io/s/pedantic-shannon-x7oku?file=/src/App.js
It doesn't happen all the time (might be a timing discrepancy as a nature of setTimeout itself) but when the slide encounters an additional render (even when the image array is consistent) the slideshow can "skip" the 5000ms duration and progress to the second slide. I've also observed this behavior clicking the right arrow right before the slide transitions (clicking once results in two slide progressions).
Might be related to commit af2fc04 and the easing addition as this didn't occur last time I used react-slideshow-image.

Thanks @jonshipman I'll look into this and get back to you

@jonshipman thanks for noticing this issue. The major culprit was this statement and also this. They are called on mouse down while inside the slide (clicking on the arrow is also considered mouse down). Once the state changes, a re-render happens and the reference to the timeout is lost so clearTimeout doesn't actually work. I've raised a PR to fix this

I've released a new version v3.3.1


⚠ Bofrost.es has degraded performance

In a0db8a6, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1563 ms

Resolved: Bofrost.es performance has improved in 52e8bee after 6 minutes.


Some house cleaning

The formatting of Cargo.tomls is my personal preference (e.g. align the equal marks, sort the items alphabetically), can revert.

Some feature questions. The icns-registrar and icns-resolver folders contain the following files:
contracts/icns-registrar
├── .cargo/
├── .github/workflows/
├── .editorconfig
├── .gitignore
├── LICENSE
└── NOTICE

If you're using Beaker, .cargo shouldn't be necessary
.github/workflows/ doesn't actually work here. Either move it to the repo's root directory or delete if not needed
.editorconfig not needed if you're not using IntelliJ. If you do it should be in the repo's root directory
.gitignore I think this should be merged with the one in the repo's root directory
LICENSE and NOTICE I think these should be placed in the repo's root directory as well

Ok if I make these changes?


change from space to ellipsis,

spaces are automatically added by fmt.Println()

Thank you @anonrose


How to register a custom cell ?

I noticed that I just can  registerNib in JSQMessagesCollectionView, is there a more flexible way to do this?
Thanks

I require a totally customized cell, but i found i need modify  JSQMessagesCollectionView to made it. I think this is not a right way.
Jes do you have some advice? Thanks very much~

Hey, have you solved this problem? I'm stuck in It.

Hey guys, here's how to do this:

Register your own cells with the collection view
Set these properties on JSQMessagesViewController:

outgoingCellIdentifier
outgoingMediaCellIdentifier
incomingCellIdentifier
incomingMediaCellIdentifier

@hit1983 - Then, you should be able to subclass JSQMessagesCollectionViewFlowLayout and set it from viewDidLoad


M3 5665 - Create page refinement

Description
Addresses Create Database feedback
Done:

At mobile screen size breadcrumb needs left margin
Add space between top of helper text and baseline of headers Number of Nodes and Access Controls to 12 px (currently 8 px)
Move tooltip about limits of selecting Linode 1 GB plan to inline with the text block
Reduce size of pricing text to 12 px (currently 14 px)
Add (required) to field label
Button to be active by default
Remove references to IP Ranges from "Add Access Control" copy
Enable SSL mode by default
Add nanode to DatabaseTypeClass type
Update is1GbPlan variable logic so that selecting the 1 GB nanode plan disables the 3 Node option (logic is currently based on memory but we're getting null back from the api rn)
remove 'DBAAS MYSQL - ' from plan labels
Make shape of tooltip more horizontal (need to double check with Matthew about break points)

TODO:

Reduce spacing between radio button and plan name to 26 px to match Linode plan table (This seems to get a little messy because of how SelectPlanPanel dynamically expands to fit available width)
[Not required] Change the hover state engine icon color to white. It currently does not change on hover and remains gray (see the gray dolphin in the attached screenshot)

Now that we surface errors more often with the action button enabled can we make these changes:
Move the plan error below its heading

Make the failover_count error look better (probably by updating packages/validation/src/databases.schema.ts with a custom message)

We'll also want to fix the failing unit test.


Some composer packages might require ssh

Is your feature request related to a problem? Please describe.
As the title said
Describe the solution you'd like
No idea if this is ideal but adding .env option ssh key location if needed
like SSH_KEY=~/.ssh/id_rsa  or absolute path /home/user/.ssh/id_rsa
AFAIK only private key is needed.
or
SSH_KEY_PRIVATE=
SSH_KEY_PUBLIC=
or just ditch the key part

Do you have an example of composer packages that require SSH keys? Is this during composer require or during the normal execution of the library the script is expecting an SSH key to be available but isn't? Just want to make sure that if it's decided to include this we have a test case to use.
In the meantime you could use a custom my_project/.warden/warden-env.yml file to mount your SSH keys:
my_project/.warden/warden-env.yml
version: "3.5"
service:
  php-fpm:
    volumes:
      - /Users/bap14/.ssh/id_rsa:/var/www/.ssh/id_rsa
  php-debug:
    volumes:
      - /Users/bap14/.ssh/id_rsa:/var/www/.ssh/id_rsa

I do think a custom mount is perhaps the most appropriate way to handle this, as there are plenty of scenarios for keeping your host keys OFF a docker image

Hi @Heisfer,
Den is using ssh agent socket forwarding for working with ssh. It works pretty good on Mac for sure because I'm using it.
If it doesn't work for you - please add more details on which OS are you, does ssh<EMAIL_ADDRESS>works for you on host machine and in side the den shell?

Error message is
error: cannot run ssh: No such file or directory
  fatal: unable to fork

I think it happens when it tries to switch branch. So I think the ssh package is missing in php-fpm?
bash: ssh: command not found
My OS is arch linux

Please try running den env pull && den env up and try again. If the issue persist - please provide your den env config | grep image

I looked at code https://github.com/swiftotter/den/blob/main/images/php-fpm/Dockerfile
apk add openssh or apk add dropbear (busybox ssh) is not in there.
So my question how should I be able to execute ssh command in den php-fpm docker?

$ den env config | grep image
image: ghcr.io/swiftotter/den-mariadb:10.4
image: ghcr.io/swiftotter/den-nginx:1.16
image: ghcr.io/swiftotter/den-php-fpm-debug:7.4-node12
image: ghcr.io/swiftotter/den-php-fpm:7.4-node12
image: ghcr.io/swiftotter/den-redis:5.0
This is just example laravel but same with magento

hm, just checked - we don't have ssh command. I think we should add it.
@navarr is there any reasons for that?

Well I realized that i could install it myself if it's needed sudo apk add openssh

hm, just checked - we don't have ssh command. I think we should add it.
@navarr is there any reasons for that?

There is no specific reason for exclusion, no

@Heisfer If you install it yourself in a running container, you'll need to do that every time you restart the container. Changes made to the file system outside of a volume (or bind mount) will not persist once the container is stopped.

@bap14 I don't believe that's true.  It won't persist beyond a den env pull or a den env down.  Only when the container is recreated, not stopped/started- but yes it's not a permanent solution.
If we feel like this is a good addition, I'll accept a PR/make the change myself on Monday

Tested pull requrest with older php version and works.

Thanks for contributing the fix!


RestClient Plugin. First commit. Solution template and Interface.

I started creating a RestClient plugin.
I created the solution with your Visual Studio template and added a method to the interface to send a request.
If you think it's a good idea I will implement it for the different platforms.

@alesvi I don't actually accept new plugins in this repo as these are the ones that I created and maintain with the community. I would recommend creating your own Repo to work out of :)


fix refresh of content while typing

In order to prevent the weird refresh behavior where the cursor would move to the beginning of the text while a user is typing, I added 2 new variables:

time last save
time last updated

Before refreshing the content of the field, I compare the time last save with the time last updated to prevent refresh from Appian to override the data while the user is typing.

@ddtobiasz @jamescarter
Could you please review this change?

@sylvainfurt I'll defer to @ddtobiasz as I don't really understand the debouncing stuff. However I recommend removing unused code that's commented out (console.log and other sections commented out)

@sylvainfurt I'll defer to @ddtobiasz as I don't really understand the debouncing stuff. However I recommend removing unused code that's commented out (console.log and other sections commented out)

Thanks James. I can remove the comments that are superfluous.
@ddtobiasz do you know who can merge this to master? FRB needs this fix and I would like to provide it to them ASAP.
Thanks!

@sylvainfurt 3 things:
1 - Please remove all the console.log comments that are all throughout this PR

2 - Both of the RTE components have the same logic with the "auto-save" or whatever we call this. So we want to keep that consistent. If you need this ASAP for someone I can put the changes in the other component afterwards though.
3 - I don't quite get what this is doing, conceptually, which I pinged you about separately.

@sylvainfurt Thanks for explaining what this change is for. Posting below for posterity.
After reviewing the other component (richTextField) with this context, I actually don't think this same issue happens there. It appears that there we use isQuillActive to avoid the problem you described.
Can you just remove all the commented out console.log (and the giant else block that's commented out) and then I'll approve!

@ddtobiasz I removed the console.logs. Please let me know of any other changes.


when use govendor async_producer.go:177:12: undefined: RecordHeader #1246

I used govendor (v1.0.9) for my project, when use go build, it reported:
vendor/github.com/Shopify/sarama/async_producer.go:177:12: undefined: RecordHeader
but if I rename vendor to vendor.bak
it will use $GOPATH, then it will be ok, I'm confused

You have the incorrect version in your vendor folder. Update it.
On Sat, Dec 29, 2018, 09:39 usernameisnull<EMAIL_ADDRESS>wrote:

I used govendor (v1.0.9) for my project, when use go build, it reported:
vendor/github.com/Shopify/sarama/async_producer.go:177:12: undefined:
RecordHeader
but if I rename vendor to vendor.bak
it will use $GOPATH, then it will be ok, I'm confused
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kardianos/govendor/issues/423, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuFsRanKxZe_Th1uQLiYfVuEbXEn1fUks5u96jBgaJpZM4Zkpw2
.

You have the incorrect version in your vendor folder. Update it.

i delete the vendor directory
and run the command:
govendor init
govendor add +e
go build
the same errror, I missed something?

[root@localhost sarama]# cat /etc/redhat-release
CentOS Linux release 7.6.1810 (Core)
[root@localhost sarama]# go version
go version go1.9.7 linux/amd64
and I compared the two files in my project and in the $GOPATH, they are the same
[root@localhost sarama]# sha1sum async_producer.go
75dd8c89fab8478c451d2f71da61352395de62ca  async_producer.go

maybe:
govendor remove github.com/Shopify/sarama/...
govendor add -tree github.com/Shopify/sarama

?


[3.0.0] URL is not a constructor

I upgraded from the previous 2.x to 3.0.0 and started getting errors in my app:
import normalizeUrl from 'normalize-url';
normalizeUrl(...);

Error:
URL is not a constructor

What node version are you using?

@joncursi can't seem to be able to reproduce the error. Any particular details?
~/P/normalize-url $ node -v
v10.2.1
~/P/normalize-url $ npm test

><EMAIL_ADDRESS>test /home/mariosant/Projects/normalize-url
> xo && ava

  index.js:8:18
  ⚠  8:18  Arrow function has a complexity of 22.  complexity

  1 warning

  9 passed

See: https://github.com/webpack/node-libs-browser/issues/69
I guess we can do conditional checking here as URL is a global in Node.js 10 too.


Fetching Data for Pages - Implement the Post Page - Q/A problem

On the https://learnnextjs.com/basics/fetching-data-for-pages/fetching-data-for-pages lesson, the question at the end asks whether the login message was seen on terminal, browser, both, or neither.  On my machine, no matter what I do, I only see the message in the terminal (and then I only see it after a 404 and a very bloody long wait - between 3-15 seconds), but the "correct answer" says I should only be seeing it in the browser.
https://github.com/arunoda/learnnextjs-content/blob/master/content/1-basics/7-fetching-data-for-pages.js#L160
  "dependencies": {
    "express": "^4.15.3",
    "isomorphic-unfetch": "^1.0.0",
    "next": "^2.4.4",
    "react": "^15.6.1",
    "react-dom": "^15.6.1"
  }

Here's a repo with the code - https://github.com/metasean-toys/next-js-tut/tree/next-tutorial.
On the next-tutorial branch, run npm run dev to see what's happening.

I had the same problem. Fixed by using Next ^4.2.0 instead of ^5.0.0.


Create tvOS Example app

I'm not sure about how out of the box this lib will work on other platforms, but it's a good idea to give it a try already.

 If you need any help just let me know


bpf: Remove duplicate command from init.sh

setup_dev already brings the interface up, so there is no need to repeat that command.

test-me-please

test-me-please

test-me-please


meilisearch: 1.3.1 -> 1.5.0

Description of changes
Release announcements:

https://blog.meilisearch.com/meilisearch-1-5/
https://blog.meilisearch.com/v1-4-release/

1.4 contains a breaking changes, but shouldn't affect most people.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

1 package blacklisted:

nixos-install-tools

1 package built:

meilisearch

Priorities
Add a :+1: reaction to pull requests you find important.

Thank you for this!


Raise permission error when config file isn't readable

Closes #3858

I'm surprised the aligned case statement passes RuboCop :P
@homu r+

:pushpin: Commit a6a343d has been approved by indirect

@homu r-

lol good catch

@homu r+

:pushpin: Commit 411d6f1 has been approved by indirect

:hourglass: Testing commit 411d6f1 with merge 26f13d1...

:sunny: Test successful - status


fix: connection timeout and not authenticated requests

Writing to info.conn before the err != nil check was causing the interface (Connection, nil) to be assigned. On subsequent nil checks due to the retry mechanism's info.conn == nil was returning false, and info.conn.IsConnected() was raising a "nil dereference error".
If a user and password are not provided and the server requires authentication, info.conn.Login() does not return an error.  Instead, an error is returned from the InfoReequest() call.
Some tests were added for these cases

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (f484524) 33.58% compared to head (06de591) 33.81%.

Files
Patch %
Lines

info/as_parser.go
90.90%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   33.58%   33.81%   +0.23%
==========================================
  Files          19       19
  Lines        4758     4764       +6
==========================================
+ Hits         1598     1611      +13
+ Misses       3052     3046       -6
+ Partials      108      107       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Readable

A test flow should be readable in a repository. It should not require a UI to run the tests. It should be succinct and compact.
Reference ansible uri and strest for examples.
Handling loops, if conditions, and wait should also be possible without requiring the tester to write JavaScript.

I didn't get what you meant. Please elaborate what you tried to convey. I would like to hear what you've to share.

one of the challenges with Postman is that it requires a UI to run and update tests.  The json output of a postman collection is unreadable because it is too complex.  because of this complexity, when these json collections are committed to a git repository they are essentially binaries that require a UI.
A better solution is to use YAML that implements a DSL for executions.  This eliminates the need for writing javascipt in pre-request scripts and tests
Ansible URI and stREST are examples of a yaml syntax with a DSL for executions.

Thanks for you quick clarification. I'll look into it.

#209 fixes this, hence closing.

This project has implemented all the things I was suggesting:
Yaml, cache, var storage, test assertion
https://testmace.com/


Add new constants to GL.hx

While trying to compile Heaps, I ran into an issue where some GL constants were missing. Not sure if this is something interesting to add to Lime or if this should be added to Heaps itself.

Lime 4 should support these using the gl.getExtension API :smile:


Updated ELASTICSEARCH_USERNAME to elastic

kibana failed to login due to typo in Elastic search username. Changing to elastic fixed the issue.

CLA signed


Installed x64 CLI, get error no TLS CA cert bundle in Program Files (x86)

Describe the bug
I just installed azure-cli-2.53.1-x64.msi on a Windows 10 system with no Azure CLI install.
The path C:\Program Files\Microsoft SDKs\Azure\CLI2\Lib\site-packages\certifi correctly contains cacert.pem.
Now I open a prompt and type az --version
I get this - note how the Program Files (x86) path is checked, not Program Files.
➜ az --version
Could not find a suitable TLS CA certificate bundle, invalid path: C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\Lib\site-packages\certifi\cacert.pem
The command failed with an unexpected error. Here is the traceback:
Could not find a suitable TLS CA certificate bundle, invalid path: C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\Lib\site-packages\certifi\cacert.pem
Traceback (most recent call last):
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 225, in invoke
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/init.py", line 121, in show_version
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/util.py", line 372, in get_az_version_string
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/util.py", line 316, in _update_latest_from_github
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/util.py", line 861, in check_connectivity
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/sessions.py", line 624, in head
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/sessions.py", line 589, in request
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/sessions.py", line 703, in send
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/adapters.py", line 458, in send
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\requests/adapters.py", line 261, in cert_verify
OSError: Could not find a suitable TLS CA certificate bundle, invalid path: C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\Lib\site-packages\certifi\cacert.pem
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
Related command
az --version
Errors
Could not find a suitable TLS CA certificate bundle, invalid path: C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\Lib\site-packages\certifi\cacert.pem
Issue script & Debug output
N/A
Expected behavior
az --version shows me expected Azure CLI and extension versions.
Environment Summary
az --version will not run. See above.

Additional context
No response

Thank you for opening this issue, we will look into it.

Hi @plzm Find similar issue #10079.
Issue title	OSError: Could not find a suitable TLS CA certificate bundle, invalid path: C:\WINDOWS\System32\cacert.pem
Create time	2019-07-27
Comment number	9
Possible solution: It seems that the requests module is not reading from the correct CA bundle path. The solution provided in issue #10079 is to check if the REQUESTS_CA_BUNDLE environment variable is set to C:\WINDOWS\System32\cacert.pem by running SET in the command prompt. If it is set, remove the environment variable and retry the az login command. In this case, the error message is slightly different, but the root cause is the same - the requests module is not finding the correct CA bundle. Therefore, the solution provided in issue #10079 should also work for this issue.
Please confirm if this resolves your issue.

Per that issue, I deleted the REQUESTS_CA_BUNDLE environment variable, started a new shell, verified the env var was gone, then tried az --version again. Now it works as expected.
After that, again started new shell. Verified REQUESTS_CA_BUNDLE was not re-created.
I was then able to successfully az ogin. After doing so, again started a new shell. Verified REQUESTS_CA_BUNDLE was still not re-created.

I suppose I will close this, since deleting the env var fixed it. Curious though why if I only have the x64 CLI installed, the env var was created pointing to Program Files (x86) instead of Program Files.