Webpack2 / postcss config question

I'm trying to apply postcss plugins to the css files specified in my my styleUrl's via postcss-loader. Their project has the following example for webpack2 config:
     {
        test: /\.css$/,
        use: [
          'style-loader',
          {
            loader: 'css-loader',
            options: { importLoaders: 1 }
          },
          {
            loader: 'postcss-loader',
            options: {
              plugins: function () {
                return [
                  require('precss'),
                  require('autoprefixer')
                ];
              }
            }
          }
        ]
      }

My understanding of angular2-template-loader is that it basically turns styleUrls: ['some-file.css'] into  styles: require('some.file.css'), which ultimately gets handled by some css loader.
Is that correct? The reason I'm questioning this is :

I tried to swap outstyle-loader with css-to-string loader in the config pasted above, but it doesn't seem to work (doesn't autoprefix the css).
The end of the README  says "The generated require statements", (for templateUrl and styleUrls), "will be handled by the given loader for .html and .js files".  I would expect that to say "for .html and .css files". Does this mean that the styleUrl generates some sort of .js file which isn't handled by a .css loader?

I asked about the setup on the postcss-loader repo (https://github.com/postcss/postcss-loader/issues/113) but they were understandably unfamiliar with this particular loader and how it should integrate with other loaders. Is there anything obviously wrong with my approach or understanding of how this loader works? Thanks!

I got postcss working by using the following config for my production build, using following package versions:
{
"autoprefixer": "6.6.1",
"angular2-template-loader": "0.6.0",
"postcss-loader": "0.9.1",
"raw-loader": "0.5.1",
"webpack": "2.1.0-beta.27",
}

// necessary imports
const autoprefixer = require('autoprefixer');
const webpack = require('webpack');
const postcssDiscardComments = require('postcss-discard-comments');

// relevant rules
rules: [{
{
      test: /\.css$/,
      loaders: ['raw-loader', 'postcss-loader']
    }, {
      test: /.html$/,
      loaders: [
        'html-loader'
      ]
}]

// add to plugins array
 new webpack.LoaderOptionsPlugin({
    debug: false,
    options: {
      htmlLoader: {
        // minimize: false // workaround for ng2
        // see https://github.com/angular/angular/issues/10618#issuecomment-250322328
        minimize: true,
        removeAttributeQuotes: false,
        caseSensitive: true,
        customAttrSurround: [
          [/#/, /(?:)/],
          [/\*/, /(?:)/],
          [/\[?\(?/, /(?:)/]
        ],
        customAttrAssign: [/\)?\]?=/]
      },
      postcss: [
        autoprefixer,
        postcssDiscardComments
      ]
    }


Bato.to extension won't update

Source information
Bato.to
Source language
English and English (United States)
Steps to reproduce

App has (1) badge on Browse tab, indicating extension update available.
On Browse tab, Extensions page has (1) badge, and lists Bato.to as Update Pending
Clicked download update next to Bato.to and allowed Android system to Install when automatically prompted.
After clicking install, screen automatically returns to Extensions page, however nothing changes. Bato.to remains as Update Pending, despite restarting app, and even restarting phone. Multiple download attempts tried over three day period.

Expected behavior
The update should install, the app should recognize extension as updated and remove from Updates Pending list. The "couldn't find script with image data" error for Bato.to files should resolve with the applied update.
Actual behavior
App indicates there is an update available. Reddit thread indicated that this update should resolve "couldn't find script with image data" error gor Bato.to files, however despite going through normal install procedure, no change takes place. Extension remains in Updates Pending list, and no errors are resolved.

Tachiyomi version
App version: 0.15.3 (standard, a9c7cbf, 119, 2024-01-13T19:03Z)
Android version
Android version: 7.1.1
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] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

/blurb uninstall


Convert wp-smiley to real emoji

It seems like WordPress RSS feeds convert emoji (for example, 📰 becomes <img alt="📰" src="https://s.w.org/images/core/emoji/13.0.1/72x72/1f4f0.png" class="wp-smiley">). Since the emoji seem to be Twitter emoji, and since NNW only works on platforms that support emoji, it would make sense to turn these into real emoji by replacing the img with its alt text.

Mapping all the WordPress images to standard emoji would be a monumental task with very little reward. This simply isn't something that we should devote developer time to even if we had it. It is really up to WordPress to straighten this situation out.

You don’t need to map the images — each image has the real emoji as its alt text, so you only need to replace the image with its alt text.


Aerail show_guides desn't work in golang

Describe the bug
When I trigger aerial in golang, "show_guide" config may doesn't work
System information

OS: [windows/linux/mac]
Neovim version:

NVIM v0.8.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by brew@Big-Sur

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.8.1/share/nvim"

Run :checkhealth for more info

AerialInfo: [output of :AerialInfo]

Aerial Info
-----------
Filetype: lua
Configured backends:
  treesitter (supported)
  lsp (supported)
  markdown (not supported) [Filetype is not markdown]
  man (not supported) [Filetype is not man]
Show symbols: Class, Constructor, Enum, Function, Interface, Module, Method, Struct

Aerial config:

                show_guides = true,

                -- Customize the characters used when show_guides = true
                guides = {
                        -- When the child item has a sibling below it
                        mid_item = "├─",
                        -- When the child item is the last in the list
                        last_item = "└─",
                        -- When there are nested child guides to the right
                        nested_top = "│ ",
                        -- Raw indentation
                        whitespace = "  ",
                },

To Reproduce
Steps to reproduce the behavior:
1.
2.
3.
If possible, provide a minimal file that will trigger the issue:
AerialOpen

Screenshots

Additional context
Add any other context about the problem here.

That's a byproduct of the structure of Go code. It's very flat; not much nesting. If there's no nesting, then there's no guides.

@stevearc

Is aerial can support display methods of receiver like the picture in golang?

done


Remove Subscribe to Newsletter function

We dont need this.

https://github.com/KHS-Minithon/website/commit/e9bbe59fbc8b0ad3e99d9df5a5818a7a1cd233fe


Fix signalling compatibility with current Jitsi Meet

Implemented necessary signalling for compatibility with Jitsi's multi-stream changes
Update deps
Switch rustls crypto backend to aws-lc-rs

Does this by any chance fix this intermittent error that I've seen recently, while running against current Jitsi meet?
2024-05-14T14:20:28.692886Z  WARN unknown payload type: 41
thread '<unnamed>' panicked at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/glib-0.17.10/src/object.rs:2590:21:
Signal 'request-pt-map' of type 'GstRtpBin' required return value of type 'GstCaps' but got None


Update sbt-scoverage to 1.9.0

Updates org.scoverage:sbt-scoverage from 1.8.2 to 1.9.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.8.2).
You might want to review and update them manually.
project/plugins.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, semver-minor, old-version-remains

Codecov Report

Merging #184 (de373de) into develop (ece30ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #184   +/-   ##
========================================
  Coverage    50.58%   50.58%
========================================
  Files            5        5
  Lines           85       85
========================================
  Hits            43       43
  Misses          42       42

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 ece30ec...de373de. Read the comment docs.


support source compression

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

@hbzhxying, could you please take a look at the prettier error?  Please follow https://aka.ms/AA6h31t to fix the issue.


Allow freestyle-logging to capture source file information(class name,line number(?)...) and pass to the logging handler

Possibly using https://github.com/lihaoyi/sourcecode#logging

Dup of #36


Upgrade to isort version 5

Description
A new major version of isort is out. This PR upgrades to it and fixes the new issues that found that the previous version had missed.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #123   +/-   ##
=======================================
  Coverage   97.17%   97.17%
=======================================
  Files          21       21
  Lines        1309     1309
  Branches      215      215
=======================================
  Hits         1272     1272
  Misses         21       21
  Partials       16       16

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 d089be9...0be2718. Read the comment docs.


Save Crop doesn't work

On staging, when editing a crop, pressing the save button doesn't do anything
/api/v1/crops/5493d4776365320002040000/
application-69eca05b90764ed6a3659913bed51d31.js:4 PUT http://staging.openfarm.cc/api/v1/crops/5493d4776365320002040000/ 422 (Unprocessable Entity)t @ application-69eca05b90764ed6a3659913bed51d31.js:4p @ application-69eca05b90764ed6a3659913bed51d31.js:4$get.a @ application-69eca05b90764ed6a3659913bed51d31.js:4(anonymous function) @ application-69eca05b90764ed6a3659913bed51d31.js:5$get.c.$eval @ application-69eca05b90764ed6a3659913bed51d31.js:5$get.c.$digest @ application-69eca05b90764ed6a3659913bed51d31.js:5$get.c.$apply @ application-69eca05b90764ed6a3659913bed51d31.js:5(anonymous function) @ application-69eca05b90764ed6a3659913bed51d31.js:6J.event.dispatch @ application-69eca05b90764ed6a3659913bed51d31.js:2J.event.add.m.handle @ application-69eca05b90764ed6a3659913bed51d31.js:2
application-69eca05b90764ed6a3659913bed51d31.js:5 ReferenceError: code is not defined
    at application-69eca05b90764ed6a3659913bed51d31.js:18
    at application-69eca05b90764ed6a3659913bed51d31.js:4
    at application-69eca05b90764ed6a3659913bed51d31.js:5
    at c.$get.c.$eval (application-69eca05b90764ed6a3659913bed51d31.js:5)
    at c.$get.c.$digest (application-69eca05b90764ed6a3659913bed51d31.js:5)
    at c.$get.c.$apply (application-69eca05b90764ed6a3659913bed51d31.js:5)
    at l (application-69eca05b90764ed6a3659913bed51d31.js:4)
    at v (application-69eca05b90764ed6a3659913bed51d31.js:4)
    at XMLHttpRequest.x.onreadystatechange (application-69eca05b90764ed6a3659913bed51d31.js:4)(anonymous function) @ application-69eca05b90764ed6a3659913bed51d31.js:5$get @ application-69eca05b90764ed6a3659913bed51d31.js:4(anonymous function) @ application-69eca05b90764ed6a3659913bed51d31.js:5$get.c.$eval @ application-69eca05b90764ed6a3659913bed51d31.js:5$get.c.$digest @ application-69eca05b90764ed6a3659913bed51d31.js:5$get.c.$apply @ application-69eca05b90764ed6a3659913bed51d31.js:5l @ application-69eca05b90764ed6a3659913bed51d31.js:4v @ application-69eca05b90764ed6a3659913bed51d31.js:4x.onreadystatechange @ application-69eca05b90764ed6a3659913bed51d31.js:4

@roryaronson I did a part-fix of this in e78288a which should at least tell us what the actual error is (the error reported is a JS error saying that the error try-catch ran but is broken :P). Being pushed live now.

I'm not getting an error on this for crops anymore, but I may have been doing this on a different crop from you, so different errors - I used cherry-tomatoes and added an image, changed some stats, etc.

Yea this looks fixed on staging. Thanks!


Deleting extra characters while extension is enabled

Noticed this behavior which I assume is unintentional.
Repro steps:

Write some text in a new email in gmail. Hit return once or twice, type "thanks " on its own line
Hold CMD and press Delete (probably works on PC with ctrl-backspace)

Expected result: Delete the entire current line and leave cursor at the beginning of the current line (ie. cmd-delete does not change the line that the cursor is on.
Actual result: When the plugin is enabled, cmd-delete deletes the entire line and then moves the cursor up to the end of the previous text, blowing away any whitespace in between.
Disabling the plugin and refreshing the page fixes the issue.

This is definitely a bug.  It's been annoying me, too.  I haven't had a chance to fix it yet but I did some diagnosis work to pinpoint it.  Adding implementation notes below.

The problem is in the Caret.js library that is used to maintain the cursor position after the highlighting is applied.  The getPos function creates a selection from the start to the cursor position, converts that to a string, and then returns the length of that string to get the position of the cursor.  The string conversion strips out any line breaks, so the number of characters passed to the setPos function is wrong in cases with whitespace.
One possible fix could be to rewrite the getPos function so that it walks the nodes of the DOM instead of doing a string conversion.
FYI, JustNotSorry is currently using my fork of Caret.js that enables use of the setPos function with ContentEditable form fields.


Attempting to register step with id extensionsWeb twice

Testing #226399
  ERR Attempting to register step with id extensionsWeb twice. Second is dropped.: Error: Attempting to register step with id extensionsWeb twice. Second is dropped.
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2403:24992
    at Array.forEach (<anonymous>)
    at Aat._registerWalkthrough (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2403:24954)
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2403:18421
    at Array.forEach (<anonymous>)
    at Aat.S (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2403:18395)
    at new Aat (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2403:18383)
    at hmt.o (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1153:1589)
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1153:3309
    at Nae.d (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:29:78918)
    at get value (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:29:79059)
    at Object.get (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1153:3832)
    at new $G (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2715:8012)
    at hmt.o (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1153:1589)
    at hmt.createInstance (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1153:1086)
    at kq.instantiate (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:413:68879)
    at H7e.Q (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:504:75630)
    at H7e.P (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:504:75412)
    at H7e.O (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:504:75173)
    at H7e.L (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:504:74607)
    at H7e.openEditor (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:504:73736)
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:616:23980
    at XF.Kb (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:616:24150)
    at XF.Jb (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:616:23753)
    at XF.openEditor (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:616:22630)
    at nje.openEditor (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1077:33690)

I see this error only in exploration build.

/duplicate https://github.com/microsoft/vscode/issues/226782


fix: flaky test in MultiValuedMap.toString

About
This PR fixes a flaky test brought by the toString method applied to the MultiValuedMap class.
What this PR is doing and why
The toString of the MultiValuedMap class is designed to produce elements printed in non-deterministic but still readable order. To preserve the semantics while removing the flakiness, this PR simply enumerates the output space and lists them as correct answers. That being said, as long as the printed output of  MultiValuedMap is one of the listed possibilities, the test can pass.

Your fix looks too big, do you even need different orders of array elements?

IMO it depends on the semantics.

If the semantic of toString requires determinism, we should fix the implementation by providing a global order to the elements.
If determinism is not desirable, we should fix the tests like what I am doing here.

To me, a typical set or map can produce elements in arbitrary order by definition. Fixing the tests by describing the output space is then desirable, despite the missing elegance of the code.
I was using an enumerator in Python to synthesize all conditions. Nonetheless, to simplify code one may implement the enumeration in Java. More concretely, something like:
@Test
public void testToString() {
    if (!isAddSupported()) {
        return;
    }

    final MultiValuedMap<K, V> map = makeObject();
    map.put((K) "A", (V) "X");
    map.put((K) "A", (V) "Y");
    map.put((K) "A", (V) "Z");
    map.put((K) "B", (V) "U");
    map.put((K) "B", (V) "V");
    map.put((K) "B", (V) "W");

    List<String> permutations = new ArrayList<>();
    for (List<Character> e : permute("XYZ", 3)) {
        for (List<Character> c : permute("UVW", 3)) {
            String permutation = "{B=" + c + ", A=" + e + "}";
            permutations.add("{B=" + c + ", A=" + e + "}");
            permutations.add("{A=" + e + ", B=" + c + "}");
        }
    }

    assertTrue(permutations.contains(map.toString()));
}

Even let's look at the HashSet in Java: https://docs.oracle.com/javase/8/docs/api/java/util/HashSet.html

The toString method inherits from java.util.AbstractCollection.toString which simply iterates the container to emit a string.

https://docs.oracle.com/javase/8/docs/api/java/util/AbstractCollection.html#toString--

The string representation consists of a list of the collection's elements in the order they are returned by its iterator, enclosed in square brackets ("[]"). Adjacent elements are separated by the characters ", " (comma and space). Elements are converted to strings as by String.valueOf(Object).

The iterator of HashSet  has no official guarantee of determinism.

Therefore, we have:

https://www.reddit.com/r/java/comments/s84ifr/i_just_realised_the_tostring_contract_does_not/
https://stackoverflow.com/questions/2704597/iteration-order-of-hashset

test1

test3


Install instructions

I think the install instructions are deprecated, at least most of the commands won't work in a newer Ubuntu release.
Can somebody update them or help me get this thing running?
I hope to get this running, sounds very very cool.

I once fought my way through all the issues and tried a lot. The documentation of the project is not easy for beginners.
I'm running an Ubuntu 20.04 VM and so far I've done the following:
Install dependencies:
Sudo apt-get install -y x11vnc xvfb sox lame qdbus-qt5 ffmpeg pulseaudio dbus-x11 xinit git maven
Install JDK 8 as described here:
https://docs.datastax.com/en/jdk-install/doc/jdk-install/installOpenJdkDeb.html
Install Spotify:
curl -sS https://download.spotify.com/debian/pubkey_0D811D58.gpg | sudo apt-key add -
echo "deb http://repository.spotify.com stable non-free" | sudo tee /etc/apt/sources.list.d/spotify.list

sudo apt-get update && sudo apt-get install spotify-client

git clone https://github.com/maximilianvoss/spotify-toniebox-sync.git

ln -sf $ PWD / spotify-toniebox-sync / bin / sp / usr / local / bin
ln -sf $ PWD / spotify-toniebox-sync / bin / stream_recorder.pl / usr / local / bin /

open the spotify-toniebox-sync folder in the terminal and
mvn clean install
everything runs smoothly
https://developer.spotify.com/
create a client there. This gives you a client ID and the client secret. Now into the musicsync.properties file. Add Redirecturl at the same time.
Toniebox access data is also there
Create cache folder and assign appropriate access rights!
Mark musicsync as executable in the folder package / target / output.
I am now only attached to the apicode for spotify. With ./musicsync -–spotify-apicode, I only get the standard output ...

Usage: spotify-toniebox-sync.jar [--daemon | --input INPUT --output OUTPUT]
--daemon
Run in daemon mode to sync periodically all lists in the properties file
--input INPUT
Define an input source for a one-time run
--output OUTPUT
Define an output destination for a one-time run

Input Plugins:
--spotify-apicode
Generate an API Code URL to get attach Application to Spotify Account
--spotify-code CODE
Generate a Refresh token out of the Spotify Code

Output Plugins:
--toniebox-username USERNAME
Set the Toniebox username to login to Toniebox
--toniebox-password PASSWORD
Set the password for the Toniebox user for login
--filesystem-directory DIRECTORY
Set the output path for the files

It's getting annoying, but giving up is not.

Okay, I installed Jdk8 so far everything worked, but from then on the commands no longer work. Cannot install Spotify, etc. Are the commands new? I just want to get the Toniebox Sync right.

fixed with latest version to also support jdk11 and updated the documentation


Add support for advanced graph queries

Add support for the following advanced graph queries:

Neighborhood
Graph of Interest
Common stream

I added neighborhood, common stream, paths between and paths from to queries from graph-algos extension to SyBLaRS. I also tested these new queries and found iVis-at-Bilkent/cytoscape.js-graph-algos#4, iVis-at-Bilkent/cytoscape.js-graph-algos#5, iVis-at-Bilkent/cytoscape.js-graph-algos#6 issues in the extension side. While these issues are being fixed, I'll continue testing the them in SyBLars.


Emerald City

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.01.05. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2017.01.05

Before submitting an issue make sure you have:

[ ] At least skimmed through README and most notably FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2017.01.05
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

lonnie@lonnie-GA-78LMT-S2P:~$ youtube-dl --verbose http://www.nbc.com/emerald-city/video/the-beast-foreverprison-of-the-abject/3447126 >bugreport.txt
[debug] System config: []
[debug] User config: [u'-o', u'[%(upload_date)s][%(id)s] %(title)s (by %(uploader)s).%(ext)s', u'--external-downloader', u'aria2c', u'--external-downloader-args', u'-c -j 3 -x 3 -s 3 -k 1m']
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'http://www.nbc.com/emerald-city/video/the-beast-foreverprison-of-the-abject/3447126']
[debug] Encodings: locale UTF-8, fs UTF-8, out None, pref UTF-8
[debug] youtube-dl version 2017.01.05
[debug] Python version 2.7.12+ - Linux-4.8.0-32-generic-x86_64-with-Ubuntu-16.10-yakkety
[debug] exe versions: ffmpeg 3.0.5-0ubuntu<IP_ADDRESS>, ffprobe 3.0.5-0ubuntu<IP_ADDRESS>, rtmpdump 2.4
[debug] Proxy map: {}
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/local/bin/youtube-dl/main.py", line 19, in
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 444, in main
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 434, in _real_main
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1791, in download
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 705, in extract_info
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 763, in process_ie_result
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 694, in extract_info
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 359, in extract
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/theplatform.py", line 250, in _real_extract
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 519, in _download_webpage
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 426, in _download_webpage_handle
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 406, in _request_webpage
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2001, in urlopen
File "/usr/lib/python2.7/urllib2.py", line 435, in open
response = meth(req, response)
File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 944, in http_response
IOError: CRC check failed 0x171d8 != 0x82e5b22aL

URL is dead


Multiple upload error

Unusual problem,when i upload many files in one thread. Sometimes i have error in uploading  (example Failed to get the EPSV port from: 6351, or unknown command)
but i try it again it work fine
i have a bad issue now, - if error thread.sleep(10 seconds)
 ftpClient = new FtpClient
            {
                Host = ftpClientSettings.Host,
                Credentials = new NetworkCredential(ftpClientSettings.Login, ftpClientSettings.Password),
                EnableThreadSafeDataConnections = false,

            };

i use the win service, that each 2 hours upload files to different servers

Sorry I cannot understand your post. Unless you clarify/explain it better its impossible to solve.


feat: save benchmark result to db

What problem does this PR solve?
save benchmark result to db, so  ckb analyzer could monitor ckb-benchmark result
Problem Summary:
What is changed and how it works?
add python function to save metrics(bench result) to database
Related changes

PR to update owner/repo:
Need to cherry-pick to the release branch

Check List
Tests

No code ci-runs-only: [ quick_checks,linters ]

Release note
None: Exclude this PR from the release note.

#benchmark


TypeError: undefined is not an object (evaluating '_victoryBrushContainer.BrushHelpers.onMouseMove.cancel')

Bugs and Questions
The Problem
I am using the BrushContainer to zoom in the diagram, like its described in the tutorial. My Code looks like this:
 <VictoryChart           width={width - buttonOffset}           height={80}           scale={{x: 'time'}}           padding={{left: 10, top: 0, bottom: 40, right: 10}}           containerComponent={<VictoryBrushContainer />}></VictoryChart>
It worked fine until i upgraded to RN 0.63.4. I was able to select an area. After the upgrade, i get the following error message:
TypeError: undefined is not an object (evaluating '_victoryBrushContainer.BrushHelpers.onMouseMove.cancel')
The error occurs on both platforms. I checked the compatibility and i use the version used in the package.json from react-native.
Reproduction
As i described above, when i trie to select an area in the brush container i get the following error:
TypeError: undefined is not an object (evaluating '_victoryBrushContainer.BrushHelpers.onMouseMove.cancel')
The versions i use:
"react": "16.13.1",
"react-native": "0.63.4",
"react-native-svg": "^12.1.0",
"victory-native": "^35.3.1"
How can i fix the bug? Is this a bug related to my versions? I needed to upgrade RN because of the iOS 14 bug. Can you pls help me?

Sry my fault. 33.0.0 works fine. Newer versions are buggy.

@niklasfessler Why was this closed? To me it seems that the builds after 33 are still suffering from this same bug.
Has anyone found a fix for this?

@niklasfessler Why was this closed? To me it seems that the builds after 33 are still suffering from this same bug.
Has anyone found a fix for this?

Hey Zeyko,
im using version 33.0.0 and it´s working. So ive closed that :) Maybe you can reopen a new Bug?
Greetings Niklas

Hey Zeyko,
im using version 33.0.0 and it´s working. So ive closed that :) Maybe you can reopen a new Bug?
Greetings Niklas

That's not a solution really. Now you're stuck with an old version and not getting any bug and security fixes or new features. But sure, i can open a new issue.

That's not a solution really. Now you're stuck with an old version and not getting any bug and security fixes or new features. But sure, i can open a new issue.

Ok ok, thats a point. I will reopen this bug.
Greetings

Ok ok, thats a point. I will reopen this bug.
Greetings

It seems like versions newer than 34.3.0 have this bug.

Happening on 35.3.3 as well

Happening on 35.3.3 as well. Directly copied the code off https://formidable.com/open-source/victory/gallery/brush-and-zoom

I'm seeing this as well using the same sample as well on a newly created Expo app


possible  merge between FYPO:0000839 FYPO:0004923

FYPO:0000839 inviable elongated mononucleate aseptate cell
Definition
A cell phenotype in which a cell contains one nucleus, has no septum, is elongated, and is inviable.
has a single child
FYPO:0004923 inviable elongated mononucleate aseptate vegetative cell with mitotic cell cycle arrest
A cell morphology phenotype in which a vegetative cell is inviable contains one nucleus, has no septum, is longer than normal, and progression through the mitotic cell cycle is arrested.
Are these equivalent?
(there are no siblings)
I guess FYPO:0000839 isn't defined vegetatively but I'm sure that is how it is used?

Well spotted - FYPO:0000839 is defined logically as vegetative but the text def doesn't reflect that. I'll fix it.
I have no idea if the two terms are interchangeable in practice for pombe, although from an ontology-geeky POV FYPO:0004923 is more specific than FYPO:0000839 because FYPO:0004923 adds the "mitotic cell cycle arrest" criterion.

I don't think there is any difference for fission yeast. ....If the cell is inviable elongated it is cell cycle arrested.

I guess you could also get inviable elongated mononucleate aseptate lysed cells?
Cell lysis isn't really arrest...?
But they would probably only be annotated to inviable elongated lysed cells
(I don't think I would think to specify aseptate for these, unless it's the other way around i.e. they are septate)

true. We haven't seen an elongated lysing cell as far as I'm aware?
2 options. One is to merge. The other is to block the parent for direct annotation for pombe.

Of the two, I prefer the merge. I don't think it's all that likely that the generic cell phenotype ontology will extend to this level of detail even if it does get built, because it's just not, well, generic. And I think the do-not-annotate scheme is more suitable for terms that are so high-level that they're not informative. It's a bit silly to use do-not-annotate for a term as detailed as FYPO:0000839.

Ok, I'm not against that

I agree.
However, note that  for GO we often use "do not annotate" for terms where the annotation would be informative, but it should be possible to be more specific (1118 terms). I think the same should apply to FYPO terms, although not necessarily in this instance.

Yes, I know, and I dislike that use of the do-not-annotate flags in the ontology.

Why? it improves the annotation...

Actually I didn't even think of this as a "do not annotate flag" in the ontology. The GO ones aren't  in GO. Its just an internal soft check?

merge done

edit file: 02e53975e4138596f952e50fe8675346a0276112
release: bd7cb989ef38a30349babc1206d0fdb124313237


Add HTML editor and tip styling button for Ask CFPB Answers

Adds buttons to the RichTextField editor on Ask admin pages so content editors can continue to create tips/warnings in Ask CFPB answers.
Additions

HTML editor in Ask admin RichTextFields
Tip styling button for Ask admin RichTextFields
CSS to style tips in RichTextFields on Ask admin pages
CSS to style tips on Ask answer pages

Testing

Check out branch
Set up Ask CFPB per instructions in #2780
Run gulp
Navigate to an Ask CFPB question in the admin

Click on the Edit HTML button on the Answer field to bring up the html editor. Verify that changes made to the HTML are preserved when you save the answer and preview the associated question page
Create a test tip header & paragraph in the Answer RichTextField. Highlight the tip and click the tip styling button. Verify that the tip is wrapped in an <aside class="answer-module> & appears with the correct styling in the RichTextField and on the associated question page.

Navigate to a non-Ask page in the admin and verify that the tip and html editor buttons do not appear on RichTextFields

Screenshots
HTML editing:
Editor button

Editor view

Tip button

Notes

Eventually we want to change the way tips are created so editors don't need to apply classes to them, but for now this replicates existing functionality
We may only need one of the buttons; this PR allows content editors to assess them both on build

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

If we go this route we are committing to couple of things:

Allowing the content folks to edit the HTML directly. We typically build the patterns outside the editor.
Adding more code to modify a deprecated Halo editor. It looks like it Wagtail will begin moving to Daft.js in Wagtail 1.10 ( https://github.com/wagtail/wagtail/projects/1)

I like the overall the direction of the PR . If the content folks are good with it, then I then I am.

This is a nice implementation! I do share some of @sebworks' concerns, though.
As usual when we talk about HTML editing I point out that you can do nasty stuff like this:

I also worry that if/when we upgrade to Wagtail 1.9/1.10 we won't have sufficient testing to detect if this breaks. Is there some way to write automated tests against this functionality?
Also, minor thing, but is it possible to change the tooltip on the Tip button to be more descriptive (maybe here)?

@sebworks @chosak I think we share your concerns about HTML editing and are hoping this will be a short term solution to allow users to work with tips. In the long run, since we need to use RichTextFields for the Ask answers, we are looking at alternatives like creating a separate field for the tip content.
Also, we're including both buttons so content editors can test them on build. If the tips styling button works well enough, we might be able to remove the html editor right away.
@chosak Wagtail does enforce an HTML element whitelist for RichTextFields when the page is saved, so we should have some protection against things like script tags.
Great point about the Tip button tooltip -- I just pushed an update:

The tip button icon probably isn't very useful either -- for expediency I just chose one that was already available. If the button's functionality tests well with the content editors, I can find a more suitable icon.

After speaking with the content team I have some additional concerns:

Copy / pasting from word directly and introducing some unwanted MSO characters.
Entering in content that creates a security hole
Entering content in violation of the Design manual guidelines
Entering malformed markup with looks correct on modern browsers but breaks on older browsers.

Why not just create a modal workflow allows them to just enter the content.

@sebworks Since this is intended to be temporary, scoped to Ask CFPB, and Wagtail does element whitelisting, I don't think those concerns should be blockers.

@virginia I added tip markup cleaning to the migration script, which might reduce the need for an HTML editor. However, imperfections in the migration and other markup oddities will make the editor useful for migration troubleshooting.
@sebworks I share those concerns, but pasting from Word (or from the web) is an issue regardless of the presence of an HTML editor, and in those cases the editor makes troubleshooting more efficient.

ok, thanks @chosak @higs4281.  I would just make sure you coordinate with the content and the design team to make sure they understand the ramifications of the changes.

@higs4281 Thanks for adding the tip cleanup!
@sebworks I share your concerns about allowing html editing. I think the complexity in this comes from creating the formatted tips inside a RichTextField, but we're not yet at a place to change to another approach like separate fields. We'll work with the content editors to test the plugins on build.
I also like your idea of a modal workflow -- I'll think about that if we continue working with tips inside the RichTextFields.


modify metadata script remote invocation from mvload so that it passes the specific database.

Replace italics below with details for this issue.
The metadata processing script currently scans ALL the databases. There is so much data now that it takes a tremendous amount of processing to complete. We have a way in the metadata update code to limit the processing to only one database.
If mvload can pass the remote metadata processing script the name of the loaded database we can limit the processing to just that database.
a few hours to test the current code and a couple of hours to implement and test.
Also...
The metadata processing script also runs at night under cron. It would be good to find a way to only process the databases that have been modified since the last time the metadata processor ran.
Molly Smith
Randy Pierce
(https://github.com/dtcenter/METviewer/issues/new/choose), [METexpress]

I found this, but have not tested it or gotten any feedback on it.
t occurs to me that if we know what databases have been touched within the last day we can run the metadata on ONLY those databases from the cron.
I believe that this query ...
mysql --defaults-file=~/.my.cnf -s -e "select filename from mysql.metadata_md_table where last_update ><PHONE_NUMBER>;" | sed -e 's|./||'
returns a list of databases that have been changed since<PHONE_NUMBER> (when I ran this I chose 24 hrs earlier for the epoch)
Using this list and the database parameter on the metadata update script we can run the metadata update ONLY on the databases that have changed.
It would be something like....
mysql --defaults-file=~/.my.cnf -s -e "select filename from mysql.metadata_md_table where last_update ><PHONE_NUMBER>;" | sed -e 's|./||' | while readd
do
-c
${HOME}/.my.cnf
-u
https://metexpres.nws.noaa.gov
-d
$d
done
or something like that.


Library and design kit catalog colors

Now that libraries and design kits have moved to the catalog section, they will need unique colors to differentiate them from other assets.

figma: https://www.figma.com/file/1HS6qRdl4GVY27iiBJWLnU/Banner_Specs?node-id=0%3A1
could potentially use teal 20 for either category.
brand is bringing in some yellow. do they use yellow 20 at all?

Libraries will be Cool gray 20 with <FileBackup /> icon

Design kits will be Teal 20 with < ArtTools_01 /> icon

@kingtraceyj is this just for catalogs, or all library pages like:
https://next.carbondesignsystem.com/libraries/carbon-react
https://next.carbondesignsystem.com/libraries/carbon-react/latest/assets
https://next.carbondesignsystem.com/libraries/carbon-react/latest/pages/tutorial/overview

@mattrosno this will be for all library pages.

@kingtraceyj if all pages in a library by default look like this:

But assets in a library have their own colors like this:

Does that mean design kits in a library looks like this?

@mattrosno good question, we should keep the gray header for the design kit page that is within the library side nav.


Add forwarded_kwrestarg node to AST::Builder

This PR adds forwarded_kwrestarg node to AST::Builder.
This feature is required to resolve https://github.com/rubocop/rubocop/issues/11730.
I thought about defining a class like ForwardedKwrestargNode for forwarded_kwrestarg, but the node is actually an anonymous KeywordSplatNode so it's used.

My only hesitation is: do we want a different class, or could we not use Kwsplat?

Ah, yeah, I thought it would be better to be able to distinguish between kwsplat node type and forwarded_kwrestarg node type, but it seems pragmatic and more better to have compatibility with kwsplat_type? as you suggested. I've updated this PR.

Thank you so much! Released as 1.28.0 🚀


Machine mac-24 lost external connection phone device.

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
infra task
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
Machine mac-24 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

External connection has been restored by rebooting machine.


SR-15929: Package plugin dependency on executable product from different package doesn't find the tool

Motivation:
Fix a bug that caused a plugin with a dependency on an executable product from different package to not find the tool when looking it up with context.tool(named:).  Dependencies on executables in the same package as the plugin did work.
Modifications:

add case to handle .product() in addition to .target() for dependencies built from executables
add test fixture
add unit test

rdar://89693333

@swift-ci please smoke test


Please roll dart_style 1.0.11 into the SDK

It contains very few actual formatting changes, so it should be safe to drop in. The latest release:

Fixes a runtime error when dart_style is itself run in Dart 2.
Force splitting an empty block as the then body of an if with an else.
Uses the new lowercase Dart 2 constant names.

(It would be good to mention those in the CHANGELOG when you bring it in.)
Thanks for rolling this in!

Friendly ping, @whesse @athomas. Flutter would like this fix soon if possible.

The DEPS are now updated, but the pre-built SDK in the repo has not been rolled. Thanks for the patch, @athomas.

 When you rebuild the pre-built SDK, you'll want to use that release.
I'm updating DEPS myself: https://dart-review.googlesource.com/c/sdk/+/53161

I've already done all the work for upgrading the built-in SDK and would
just need to land it. Getting a new one means a new dev release. How urgent
is it to get the latest fix in?
On Mon, Apr 30, 2018, 23:45 Bob Nystrom<EMAIL_ADDRESS>wrote:

I found another little Dart 2 bug inside dart_style, so I kicked out
another release. When you rebuild the pre-built SDK, you'll want to use
that release.
I'm updating DEPS myself:
https://dart-review.googlesource.com/c/sdk/+/53161
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/32961#issuecomment-385538305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATEUh3pxn40_Iks26dBQcmNBQ5t98Bt5ks5tt4XmgaJpZM4Tibho
.

Not urgent at all. All that really matters is updating the DEPS so that Flutter folks can get a version of dart_style that's Dart 2 compatible.

I didn't see that the issue title was changed->reopen.

Checked in SDK is now update to 2.0.0-dev.52.0 which includes the bump 1.0.12.

Thanks!


Add optional support for tree-sitter

We can replace most, if not all, of our custom movement functions with tree-sitter, making them more robust.

Is it viable to contribute go-ts-mode to emacs itself? With eglot already part of emacs, emacs will be able to provide modern Go environment out of the box, provided gopls is available.

If you want to put up with GNU and the FSF to get your code into Emacs, feel free to. I, however, won't take part in that.
(I think there are also less political arguments against having go-ts-mode in Emacs, such as mismatching release cycles between Go and Emacs.)

Based on the document To1ne linked to, it seems like we should implement a new major mode go-ts-mode. Code that can be shared between the current go-mode and the new go-ts-mode will go in a base mode go-base-mode.
I will probably work on this myself in the near future.

FYI
Emacs has worked on a go-ts-mode, https://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-29&id=fee2efe1b035d601ac53a32801227402e9be8bca

Emacs has worked on a go-ts-mode, https://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-29&id=fee2efe1b035d601ac53a32801227402e9be8bca

That's great! With that I consider this issue closed.
One thing would be nice though, if I could use gofmt-before-save with go-ts-mode. Something along the lines of:
 (defun gofmt-before-save ()
   "Add this to .emacs to run gofmt on the current buffer when saving:
 \(add-hook 'before-save-hook 'gofmt-before-save).

 Note that this will cause ‘go-mode’ to get loaded the first time
 you save any file, kind of defeating the point of autoloading."

   (interactive)
-   (when (eq major-mode 'go-mode) (gofmt)))
+   (when (member major-mode '(go-mode go-ts-mode)) (gofmt)))

Until that, I have this in my config:
(defun my-go-mode-setup ()
  "Setup Go mode to my liking."
  (add-hook 'before-save-hook 'gofmt nil t))

(use-package go-mode
  :ensure t
  :mode (("\\.go$" . go-ts-mode))
  :hook ((go-mode go-ts-mode) . my-go-mode-setup)
  :custom (gofmt-command "goimports"))

Or you could use this if you like: https://gist.github.com/GnaneshKunal/3d3f982ce1903990eedd586952893422

That's great! With that I consider this issue closed.

I don't. Using that mode loses you most of the features of the current go-mode, and doesn't use tree-sitter to its full potential to replace most of the custom navigation functions in go-mode, like go-goto-arguments.
Big fan of the fact they took the name go-ts-mode, too, while not owning go-mode.

Closing in favour of #428.


EU GDPR Compatibility (DSGVO)

Hey, for the EU all CDN and Google fonts have to be replaced.
We started with that, but since many field types do, we can not update after that.
Are you planning that?
Greets from Germany
Philipp

Hi @PKuebler by "replace" do you mean to store all fonts localy?  If that is the case acording to this https://github.com/google/fonts/issues/1495#issuecomment-394168955 from @davelab6, wich was updated 18 days ago, google is working on it.
Woldn't be be better to wait how this is resolved by google or if they are going to resolve at all, backpack is not the only project affected by this issue, if fact I belive that 99% of project being dinamic or static are affected by this. :)
Don't take me bad, You're right, right now the Google fonts are a problem for the GPRD but maybe it's better to wait a little longer until we see how this issue is resolved by Google.

This does not only affect Google Fonts. We have to replace all external sources which are loaded via CDNs (cdn.jsdelivr.net, cdn.datatables.net, cdnjs.cloudflare.com, ...).

Well thank you EU for this... I was unaware that CNDs breach GDPR, thank you @PKuebler .
If that's the case, yes, we will probably replace all CDNs with locally-hosted files. But I agree with @androidedev  that we should wait a bit on this, see if the issue resolves itself and we can keep using CDNs.
I personally love using CDNs, and additionally use CloudFlare for most of the stuff I develop. I'd hate to reduce website performance because of this law...

Well thank you EU for this... I was unaware that CNDs breach GDPR

CDNs don't necessarily breach GDPR - a CDN is simply a service operated by a 3rd party (similarly to e.g. a web hosting) and may or may not be GDPR compliant - most of them are compliant, since they have many users in EU.
Some people just prefer reducing the number of used 3rd party services as it means a smaller risk of GDPR-related problems.

Theres issues on this within the Google Fonts repo and they confirmed it's not breach of GDPR as you're not really providing any personal information to them.
Yes the IP address of the user is revealed as they make a request - but this is deemed as acceptable as the user can choose to block this from the browser, additionally it would break 1/2 the internet.
The data is not stored for processing etc theoretically.
However, we recently put CSP in place and noticed there are A LOT of different CDNs used throughout backpack, including local scripts, so it seems a bit of a mess in that instance.
If a tidy up was due it should probably be around

Making sure all CDNs are from the same one
Make everything local

But not because of GDPR, for the sake of sanity :D

@OwenMelbz I've read abit of the google forms on GDPR relating to all their services they continue to say they don't store user data and do not but do not disclose if they store the server request or any data. they simply say "they do not store the user's IP" it creates weird gray area and concerns many people who deal with that type of data because in the end of google is breaching GDPR we don't know and the lawsuit can end up back on the company using a CDN. As open as google typically is with most info this is something they haven't been very open about.
I wonder if maybe reaching out to StackPath is an option for Backpack (@tabacitu) They host the CDN for Bootstrap, FontAwesome, jQuery, and more without cost cause they are open source. I know it is a stretch but perhaps it's something we can try and work with them on to help spread word more about Backpack, Laravel as a whole etc. (https://www.stackpath.com/open-source) I've looked into all 3 that i mentioned at least and yeah their latest CDNs are all GDPR compliant (Bootstrap 4, FontAwesome 5, jQuery 3.3)

Based off what we’re doing at work, and the fact we’re in the EU, we don’t believe there is anything to worry about.
Think of it this way:

a CDN is a host to provide website files.
a web server is a host to provide website files.

You do not run the web server, a hosting company does.
You do not run the CDN, a hosting company does.
Thus it’s treated the same, if your website was hosted on google cloud platform, you wouldn’t be asking these questions. If a user can show legitimate interest in your service you can use the data responsibly. Simply serving them a website is using doing just that.
—
But I still think we should bin off the CDN completely as it’s a mixture of different CDNs and local files. So makes sense to unify.

@tabacitu This doesn't seem to be an actual issue and should be closed, however as it may affect a whole bunch of people if it does - and I'm not the guy where the issue stops (Tabacitu Trump for the win) - I'll let you close this one off.
This should NOT become a discussion of HOW we're going to get rid of CDNs, though.
#1404 might be that one; and I'm sure there was another one at some stage.

I agree - let’s move the conversation there.


Update to breaking GitHub hooks changes

GitHub will deprecate a few things for the Webhooks in November. So we should switch to the new names:
Github replaced  integration_id with app_id
The integration_installation_repositories will be deprecated in favor of installation_repositories and integration_installation in favor of installation_repositories
See https://developer.github.com/webhooks/

should be an easy change for us. But we would need to test it in the staging system. @christophwitzko how can we access that?


Project creation refactor

opening this PR instead for it to be a draft while its in testing

So can I test this now?

@miladsoft will have to give it a touch once it is pushed

Ideally the predefined stages will be a popup.
but I think we can do that in a separate PR, if this one is good enough I would rather push it already and make a popup for predefined stages separately

I wonder if this is not a bit confusing, can the user do anything after creating the profile on Nostr?
We should at least disable all the other buttons, they get disabled if I navigate away and then back but not if I navigate to the previous tab

This is not so clear, bug I guess its is more a UI change

I wonder if this is not a bit confusing, can the user do anything after creating the profile on Nostr? We should at least disable all the other buttons, they get disabled if I navigate away and then back but not if I navigate to the previous tab

after a user creates the nostr meta data, he needs to fill in the project info and create that.
he cannot create the project yet.
although we can add that the active tab is sent to where it has been reset, but the main reason of this PR is to give the user the option to do anything he wants to do here, and finalize his project when he wants. (Depending on what we do on the fail safe i'll adjust this)

This is not so clear, bug I guess its is more a UI change

you're correct, i'll try to make it more apparent that you have 2 choices to create stages.

I realise we have 3 operations that need to happen before a project is finalized

post project metadata to nostr
post project info to nostr
post a trx to the blockchain

We should probably have 3 buttons that each becomes enabled at their turn (I think it is like that now?)
If any of the stages failed the use should be able to try again that stage until is succeeds

I noticed when I create a project we have two entries in the database for that project one is missing the trx id

I was able to add a stage after creating the profile

now I can not remove the stage the remove button seems to have disappeared.

The create button should be like the other two in my opinion
it looks nice like that

@dangershony for each different indexer it seems like the api route is different, for example mempool doesnt have "query" like angor indexer.
how do we want to address that?
possible solution is to have route normalization or api mapping.
also we can just not support it (even though it might be on the same network)
links for genesis block hashes -
angor testnet indexer link  : https://tbtc.indexer.angor.io/api/query/block/index/0
mempool testnet indexer link:  https://mempool.space/testnet/api/block-height/0
mempool signet : https://mempool.space/signet/api/block-height/0

There are conflicts, can you fix them before I dig in to this?

There are conflicts, can you fix them before I dig in to this?

merged


Error with MAPL develop and NAG 7.1_7113

I recently grabbed NAG 7.1_7113 and decided to build MAPL with it. When I did so I got the following error:
NAG Fortran Compiler Release 7.1(Hanzomon) Build 7113
Warning: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90, line 248: INT64 explicitly imported into MAPL_HEAPMOD but not used
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90: Inconsistent INTERFACE block for procedure LOADR2D from MAPL_ALLOC_R_2D
       Argument A (no. 1) should have the TARGET attribute
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90: Inconsistent INTERFACE block for procedure IVAL1 from MAPL_DEALLOC_R_2D
       Argument I (no. 1) should not be a POINTER
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90: Inconsistent INTERFACE block for procedure IVAL2 from MAPL_DEALLOC_R_2D
       Argument I (no. 1) should not be a POINTER
[NAG Fortran Compiler error termination, 3 errors, 1 warning]
make[2]: *** [shared/CMakeFiles/MAPL.shared.dir/build.make:141: shared/CMakeFiles/MAPL.shared.dir/MAPL_HeapMod.F90.o] Error 2
make[1]: *** [CMakeFiles/Makefile2:2458: shared/CMakeFiles/MAPL.shared.dir/all] Error 2
make: *** [Makefile:166: all] Error 2

So, we have:
      interface loadr2d
         function loadr2d(P1,I1,I2) result(P2)
           real          :: P1
           real, pointer :: P2(:,:)
           integer, intent(IN) :: I1, I2
         end function loadr2d
      end interface

and:
  function loadr2d(a,i,j) result(ptr2)
    implicit none
    integer, intent(IN) :: i,j
    real, target, intent(IN) :: a(i,j)
    real, pointer :: ptr2(:,:)
    ptr2 => a
  end function loadr2d

So, okay, I do what NAG says and add the target:
      interface loadr2d
         function loadr2d(P1,I1,I2) result(P2)
           real, target  :: P1
           real, pointer :: P2(:,:)
           integer, intent(IN) :: I1, I2
         end function loadr2d
      end interface

and:
[  5%] Building Fortran object shared/CMakeFiles/MAPL.shared.dir/MAPL_HeapMod.F90.o
NAG Fortran Compiler Release 7.1(Hanzomon) Build 7113
Warning: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90, line 248: INT64 explicitly imported into MAPL_HEAPMOD but not used
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90: Inconsistent INTERFACE block for procedure LOADR2D from MAPL_ALLOC_R_2D
       Argument A (no. 1) should be an array
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90: Inconsistent INTERFACE block for procedure IVAL1 from MAPL_DEALLOC_R_2D
       Argument I (no. 1) should not be a POINTER
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90: Inconsistent INTERFACE block for procedure IVAL2 from MAPL_DEALLOC_R_2D
       Argument I (no. 1) should not be a POINTER
[NAG Fortran Compiler error termination, 3 errors, 1 warning]
make[2]: *** [shared/CMakeFiles/MAPL.shared.dir/build.make:141: shared/CMakeFiles/MAPL.shared.dir/MAPL_HeapMod.F90.o] Error 2
make[1]: *** [CMakeFiles/Makefile2:2458: shared/CMakeFiles/MAPL.shared.dir/all] Error 2
make: *** [Makefile:166: all] Error 2

Ah, okay, should be an array. Well, lets make the interface nigh-identical:
      interface loadr2d
         function loadr2d(P1,I1,I2) result(P2)
           integer, intent(IN) :: I1, I2
           real, target, intent(IN) :: P1(I1,I2)
           real, pointer :: P2(:,:)
         end function loadr2d
      end interface

and:
[  5%] Building Fortran object shared/CMakeFiles/MAPL.shared.dir/MAPL_HeapMod.F90.o
NAG Fortran Compiler Release 7.1(Hanzomon) Build 7113
Warning: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90, line 248: INT64 explicitly imported into MAPL_HEAPMOD but not used
Error: /Users/mathomp4/Models/MAPL-NAG-7.1_7113/MAPL/shared/MAPL_HeapMod.F90, line 182: No specific match for reference to generic LOADR2D
[NAG Fortran Compiler error termination, 1 error, 1 warning]
make[2]: *** [shared/CMakeFiles/MAPL.shared.dir/build.make:141: shared/CMakeFiles/MAPL.shared.dir/MAPL_HeapMod.F90.o] Error 2
make[1]: *** [CMakeFiles/Makefile2:2458: shared/CMakeFiles/MAPL.shared.dir/all] Error 2
make: *** [Makefile:166: all] Error 2

because of:
      a => loadr2d(heap%buffer(heap%HP_start(i)),im,jm)

and buffer is 1-D:
     real, pointer  :: BUFFER(:)=>null()

I get the same error if I do:
           real, target, intent(IN) :: P1(:,:)

I think I will defer to @tclune at this point. Passing a 1-D array to a 2-D array is one of those things in Fortran that always confused me. :)

long-terming this for now...

Closing for now. NAG works in MAPL3 which is enough for now.

Reopening as @tclune  is looking at this


chore: add cron run to github integration job

Add an additional task to integration test to run cron and try to detect configuration issues
Is necessary to run cron after integration test because DbMetadata needs to be initialized
This is not a final solution because config can be updated in staging or prod (CI tasks need to be updated) but will help developers to avoid issues with cron job

Example

If we run it here we also need to run it in concourse - and I think the best place to test is part of the galoy smokiest that gets executed on the real cluster.

This is now being run as part of the smoketest @ staging rollout.

a rebase is needed @dolcalmi


Create drive class for this year and a driver-control program that uses it

Given the similar chassis the drive class should be similar, unless you  have good ideas for improving over last year.
You'll want to create an initial drive program for testing the chassis that uses the drive class

Jacob completed this on 10/8


WIP: Compose approach

This is a slight variation on the original skeleton, which uses docker compose to provision a Postgres container that can work as a target warehouse.
NOTE: if the POSTGRES_HOST is set to host.docker.internal then this setup would simply ignore the postgres container and use a server that the host is connected to.

Hi @boblannon-fp. I like the approach of using docker compose to make it simpler for users to get things set up with a postgres container.

Thanks! As noted, it's just a WIP, wanted to see what you thought of the general idea.  Feel free to close it if you'd rather not have a draft PR hanging around.

However, the current PR breaks some existing functionality that I'd want to see fixed.

I will look into this, but probably not very soon.  If anyone would like to branch off of this and try something different, by all means, have at it!

Also, I believe the README would need to be adjusted to support the new workflow?

Yes, definitely.


Webpack - extract separate vendors chunk for better caching

Details

Extract 3rd party dependencies (~75% of App) to separate js file for more efficient caching
3rd party deps don't change as often as main source
After App update (web) end users would download just the main source and resolve the rest from cache
When dependencies do change cache is invalidated and users download everything - same as before
Fixed Issues

$ https://github.com/Expensify/App/issues/12235
PROPOSAL: N/A
Tests

[ ] Verify you can still use development mode = npm run web
[ ] Verify you can still use desktop development = npm run desktop
[ ] Verify you can build the staging app npm run build-staging

[ ]  And you can serve it locally: npx serve -s dist

[ ] Verify you can build the prod app npm run build

[ ]  And you can serve it locally: npx serve -s dist

[ ] Verify you can build desktop staging npm run desktop-build-staging

[ ]  And you can start it locally - can be launched from ./desktop-build folder

[ ] Verify you can build the prod app npm run desktop-build

[ ]  And you can start it locally - can be launched from ./desktop-build folder

Offline tests

N/A - same offline behavior as before
QA Steps

[ ] Verify the web app still loads

[ ] And it doesn't load slower than before

[ ] Verify the Desktop app still launches without errors

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@roryabraham did you want to take a look here or good with me merging?

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by @chiragsalian in version: 1.2.38-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Replace E_NOINTERFACE magic number with constant from metadata

Just like E_POINTER this constant can be generated and used from the bindings, instead of being hardcoded across the source.

I decided not to export HRESULT constants from the Windows crate even though that would be useful to me. The Windows crate should be the bare minimum to support the definition of generated API definitions.
There's always "just one more" HRESULT that someone deems important to their project. Instead, the Windows crate is kept to a minimum and projects can define these constants for themselves using the build macro. There's also no guarantee that the definitions are even available, which is why the generated code uses the magic numbers.
Once I have pre-generated bindings working, that will become the primary source for such things.

@kennykerr Not aiming to export "just one more" HRESULT here - but replacing the magic constants used within the windows crate itself. That should be enough reason to generate a binding for them, just like E_POINTER.
If you prefer the reexports can be removed, everything within windows can still import it from bindings:: instead of redefining the constant.

There's also no guarantee that the definitions are even available, which is why the generated code uses the magic numbers.

I don't really understand what you mean here: they're in the metadata checked in to the repo, in Foundation even. That file is totally under our control and already has quite a few imports in bindings/src/main.rs. The generated result is checked in to this repository in src/bindings.rs and won't magically disappear.

There's a distinction between what is generated in src/bindings.rs purely for the internal use of the Windows crate and what is generated in windows.rs by the build macro for public consumption. While we could certainly add more HRESULT constants to src/bindings.rs for internal use, we cannot refer to those in windows.rs because they may not be available. They may not be available because src/bindings.rs and windows.rs can potentially be generated from an entirely different set of winmd files. Yes, the metadata used to build the Windows crate is known, but the metadata used to generate windows.rs is not. The build macro will use whatever metadata is in the workspace's .windows/winmd folder and only fall back to the default metadata if no Windows.* metadata is found. This allows developers to use newer or entirely different metadata for their project.

There's a distinction between what is generated in src/bindings.rs purely for the internal use of the Windows crate and what is generated in windows.rs by the build macro for public consumption. While we could certainly add more HRESULT constants to src/bindings.rs for internal use, we cannot refer to those in windows.rs because they may not be available.

Correct. I'm totally aware of this distinction, and this patch does nothing to - nor would it ever be my intention to - shove HRESULTs into windows.rs. All it does is add them to src/bindings.rs (and reuse the reexport in some crates).
I guess you want to revert the changes in the tests to use the constants from their own windows.rs bindings, and remove the reexport from HRESULT (and potential const reexport from Result?


Make it possible to skip/ignore rules/linting on specific lines

Is your proposal related to a problem?
Yes
The linting can fail on certain complex lines, like for example our (valid) usage of mail command for bareos backups in a template file:
"/usr/lib/bareos/bsmtp -h localhost -f \"\(Bareos\) \<%r\>\" -s \"Bareos: %t %e of %c %l\" %r"
Taken from: https://docs.bareos.org/Configuration/Messages.html#config-Dir_Messages_MailCommand
Describe the solution you’d like
Im not sure how to format the line above (or even if it is possible) so that the linter accepts it..
but I do know that the template and command works..
So I would like the possibility to skip this linting.
Just skipping the/a line, not skipping it for the whole 200+ lines file as linting still is good here.
Either skipping curlylint completely or just this rule.
For example with a # noqa ... line above the one you want to skip or something similar.

Actually it is the parsing of the file that fails, not any linting rule..
151:75	Parse error: expected one of '[:a-zA-Z]', 'animate', 'animateMotion', 'animateTransform', 'area', 'base', 'br', 'circle', 'col', 'ellipse', 'embed', 'feBlend', 'feColorMatrix', 'feComposite', 'feConvolveMatrix', 'feDisplacementMap', 'feDistantLight', 'feDropShadow', 'feFlood', 'feFuncA', 'feFuncB', 'feFuncG', 'feFuncR', 'feGaussianBlur', 'feImage', 'feMergeNode', 'feMorphology', 'feOffset', 'fePointLight', 'feSpotLight', 'feTile', 'feTurbulence', 'hr', 'image', 'img', 'input', 'line', 'link', 'meta', 'mpath', 'param', 'path', 'polygon', 'polyline', 'rect', 'script', 'set', 'source', 'stop', 'style', 'track', 'use', 'wbr', '{#', '{%', '{{' at 151:75	parse_error

But if the line could be skipped completely from even that it would be great!
Also some more exact error description could be nice as well.. but skipping is more important =)

Hey @tomasbackman, thank you for reporting this!
What you’re asking sounds a lot like #36.
Could you share more from your template? Just with this line alone I’m not entirely sure I understand what the parser might be failing on.

@thibaudcolas sorry for late reply, I just checked curlylint again (using version 13.0 on macos Big Sur 11.6)
And parsing fails for file containing below (a snippet of a .j2 template file for bareos configuration we are pushing out with  ansible)
...
# Message delivery -- send most everything to email address
#  and to the console
Messages {
  Name = Standard
  Mail Command = "/usr/bin/bsmtp -h localhost -f \"\(Bareos\) \<%r\>\" -s \"Bareos: %t %e of %c %l\" %r"
  Operator Command = "/usr/bin/bsmtp -h localhost -f \"\(Bareos\) \<%r\>\" -s \"Bareos: Intervention needed for %j\" %r"
  Mail =<EMAIL_ADDRESS>= all, !skipped
  Operator =<EMAIL_ADDRESS>= mount
  Console = all, !skipped, !saved
  Append = "/var/log/bareos/bareos.log" = all, !skipped
  Catalog = all
}
...

The snippet above is valid config and in production use in our bareos backup servers.
I think it is the Mail and Operator Command lines that makes curlylint fail (due to the repeated escapes and quotes maybe?). I tried to rewrite the lines a few times so that both the linting and the config worked but did not succeed in that.. :/.
What I have done is to turn of the linting of the whole file.. which I do not like, as Im a proponent for liniting and similar quality checks. So just ignoring a specific line (like adding #noqa in the end like in ansible-lint) with a mention would work nicely here I think.
Or if the linter/parsing became able to handle the line, but linting that can handle all different script syntax is pretty complex is my guess.. so my suggestion would be to first add ignore capabilities =)

Output fromm command (above snippet in a file)
$ curlylint curly-lint-test.txt
curly-lint-test.txt
5:64	Parse error: expected one of '[:a-zA-Z]', 'animate', 'animateMotion', 'animateTransform', 'area', 'base', 'br', 'circle', 'col', 'ellipse', 'embed', 'feBlend', 'feColorMatrix', 'feComposite', 'feConvolveMatrix', 'feDisplacementMap', 'feDistantLight', 'feDropShadow', 'feFlood', 'feFuncA', 'feFuncB', 'feFuncG', 'feFuncR', 'feGaussianBlur', 'feImage', 'feMergeNode', 'feMorphology', 'feOffset', 'fePointLight', 'feSpotLight', 'feTile', 'feTurbulence', 'hr', 'image', 'img', 'input', 'line', 'link', 'meta', 'mpath', 'param', 'path', 'polygon', 'polyline', 'rect', 'script', 'set', 'source', 'stop', 'style', 'track', 'use', 'wbr', '{#', '{%', '{{' at 5:64	parse_error

Oh no! 💥 💔 💥
1 error reported

We also have other occurrences in our code base where curlylint fails with similar problems in various template files (shell commands with mixes of quotes and backslashes etc.
Our code base is quite large and we have introduced linting to enhance the quality of it.. but linting not working, and especially not being able to turn off on case-by-case(line-by-line) basis is not helping us. Even if it does much other good for the files without problems.


SIGSEGV for cached, guvectorize functions compiled in object mode

If caching a guvectorize function that is compiled in object mode, upon calling the cached function, the process is terminated with a SIGSEGV (or python.exe has stopped working etc.)
For example running the following script:
from numba import double, guvectorize
import numpy as np

@guvectorize([
    (double[:], double[:]),
], '(n)->(n)', cache=True, forceobj=True)
def gufunc(x, out):
    out[:] = x + 1

if __name__ == "__main__":
    print(gufunc(np.array([1.0, 2.0, 3.0])))

The first time yields:
[ 2.  3.  4.]

But the second time yields:
... “python ...py” terminated by signal SIGSEGV (Address boundary error)

Tested on a couple of ubuntu 16.04 machines and windows, with numba 0.35.

Thanks for the report and minimal reproducing sample. I can reproduce locally, will take a better look shortly.

This does not reproduce on version 0.43.1. Closing.


ScriptHoovDotNet/RagePluginHook Issue Resolved yet?

So if this is in fact an issue with Rage Plugin Hook and it looks like thats the case, then why don't someone from this website communicate with someone over on Rage Plugin Hook to try and get this issue worked out? I just don't get why it hasn't been fixed yet. Allot of us play LSPDFR with a scripts folder therefore requiring Rage plugin Hook to run our game. So, what's the issue with getting some communication between ScriptHooVDotNet and Rage Plugin Hook?

It was solved in 29ba91ea83c9ae5b11356485faa4c063144f0903.

No it was not resolved.  This has been an ongoing issue for a VERY long time.

@LegacyEffect Have you tried the latest dev build, which does not ruin C++ exceptions and the exe keeps running when RPH plugins throw unhandled exceptions? Download the zip file of the latest build and see if the issue still happens.
All official releases (v3.6.0 or earlier) suffer the issue, but I'm thinking we will release the next one within a month.


backport nokogiri 1.11.0 to allow Ruby 2.4.5

What problem is this PR intended to solve?

Have you included adequate test coverage?

Does this change affect the behavior of either the C or the Java implementations?

Sorry, I will not support older versions of Ruby. The effort to precompile for those platforms is too great.

@flavorjones sorry about this PR - no expectation on my part that you'd support 2.4.5 any longer. I had intended to open the PR on my own fork but the GitHub UI default option sends it to the original repo.
And yes, I am trying to backport for that CVE, although it's one of dozens across many gems in a legacy application that need vuln updates. Thanks for the pointers to the mitigation suggestions.


Rename workflows so they are more readible on the GitHub Actions page

Goal

Testing

Release Notes

WHAT:
WHY:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1254  👈
#1230 : 1 other dependent PR (#1231 )
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite

Merge activity

Aug 14, 5:17 PM EDT: @bidetofevil started a stack merge that includes this pull request via Graphite.


Add a code of conduct

Consider adding a code of conduct for Perkeep.
We can either reference the existing Google code of conduct:
https://opensource.google.com/conduct/
Or consider adding our own.

Reference Google's. We don't want to write our own.

Using Google's is fine with me.
What's our governance model for decisions like this?  I guess I can use lazy consensus?
"If there are no objections by Mar 3rd, 2018 to this decision I will send a CL adding a one-paragraph section to README.md noting that we use the Google Code of conduct and directing any issues to<EMAIL_ADDRESS>

What's our governance model for decisions like this?

I don't know. We don't have one. BDFL? Lazy consensus works usually, but for things like CoC I'm fine with just decreeing that we have one.

"If there are no objections by Mar 3rd, 2018 to this decision I will send a CL adding a one-paragraph section to README.md noting that we use the Google Code of conduct and directing any issues to<EMAIL_ADDRESS>
Shouldn't that be sent to the ML as well? Just to be clear, I don't care. Just a remark/question.

https://camlistore-review.googlesource.com/c/14686/
Email sent to mailing list with lazy consensus for Mar 19.

My only concern with the patchset is that it does not name a Project Steward and contact information. Presumably<EMAIL_ADDRESS>is not the appropriate point of contact for folks who wish to report code of conduct-related problems to perkeep.

The Google Opensource folks are willing to intervene on these matters at
this time.  The Perkeep project currently falls under the Google umbrella.
On Wed, Mar 14, 2018 at 12:03 PM Paul Ivanov<EMAIL_ADDRESS>wrote:

My only concern with the patchset is that it does not name a Project
Steward and contact information. Presumably<EMAIL_ADDRESS>is not
the appropriate point of contact for folks who wish to report code of
conduct-related problems to perkeep.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/perkeep/perkeep/issues/1070#issuecomment-373138965,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF5ywR0td5i5Q48abNYA4VlBQy-_ruVks5teWlsgaJpZM4SXTpu
.

https://camlistore-review.googlesource.com/c/camlistore/+/14726

<EMAIL_ADDRESS>comes to me (and a few others), so if I ever got any reports, I'd end up reaching out to Brad and company anyway.


Improving layout of grammar rules (sqlparse.yrl)

Today's grammar rules consist of two parts (rule and processing instruction) in equally wide columns as follows:
query_partition_clause -> PARTITION BY     scalar_exp_commalist                                 : {partition_by, '$3'} .
query_partition_clause -> PARTITION BY '(' scalar_exp_commalist ')'                             : {partition_by, '$4'}.

outer_join_type -> FULL                                                                         : full.
outer_join_type -> FULL  OUTER                                                                  : full_outer.
outer_join_type -> LEFT                                                                         : left.
outer_join_type -> LEFT  OUTER                                                                  : left_outer.
outer_join_type -> RIGHT                                                                        : right.
outer_join_type -> RIGHT OUTER                                                                  : right_outer.

table_ref -> table                                                                              : '$1'.
table_ref -> query_term                                                                         : '$1'.
table_ref -> query_term NAME                                                                    : {as, '$1', unwrap_bin('$2')}.

join_ref -> table                                                                               : '$1'.
join_ref -> query_term                                                                          : '$1'.
join_ref -> query_term NAME                                                                     : {as, '$1', unwrap_bin('$2')}.

hierarchical_query_clause -> START WITH search_condition CONNECT BY         search_condition    : {'hierarchical query', {{'start with', '$3'}, {'connect by', <<>>, '$6'}}}.
hierarchical_query_clause -> START WITH search_condition CONNECT BY nocycle search_condition    : {'hierarchical query', {{'start with', '$3'}, {'connect by', '$6', '$7'}}}.
hierarchical_query_clause -> CONNECT BY         search_condition START WITH search_condition    : {'hierarchical query', {{'connect by', <<>>, '$3'}, {'start with', '$6'}}}.
hierarchical_query_clause -> CONNECT BY nocycle search_condition START WITH search_condition    : {'hierarchical query', {{'connect by', '$3', '$4'}, {'start with', '$7'}}}.

This is a usability problem for narrow screens.

Suggested solution: The column width is fixed by rule block as follows:
query_partition_clause -> PARTITION BY     scalar_exp_commalist     : {partition_by, '$3'} .
query_partition_clause -> PARTITION BY '(' scalar_exp_commalist ')' : {partition_by, '$4'}.

outer_join_type -> FULL        : full.
outer_join_type -> FULL  OUTER : full_outer.
outer_join_type -> LEFT        : left.
outer_join_type -> LEFT  OUTER : left_outer.
outer_join_type -> RIGHT       : right.
outer_join_type -> RIGHT OUTER : right_outer.

table_ref -> table           : '$1'.
table_ref -> query_term      : '$1'.
table_ref -> query_term NAME : {as, '$1', unwrap_bin('$2')}.

join_ref -> table           : '$1'.
join_ref -> query_term      : '$1'.
join_ref -> query_term NAME : {as, '$1', unwrap_bin('$2')}.

hierarchical_query_clause -> START WITH search_condition CONNECT BY         search_condition : {'hierarchical query', {{'start with', '$3'}, {'connect by', <<>>, '$6'}}}.
hierarchical_query_clause -> START WITH search_condition CONNECT BY nocycle search_condition : {'hierarchical query', {{'start with', '$3'}, {'connect by', '$6', '$7'}}}.
hierarchical_query_clause -> CONNECT BY         search_condition START WITH search_condition : {'hierarchical query', {{'connect by', <<>>, '$3'}, {'start with', '$6'}}}.
hierarchical_query_clause -> CONNECT BY nocycle search_condition START WITH search_condition : {'hierarchical query', {{'connect by', '$3', '$4'}, {'start with', '$7'}}}.

Solved by PR #100.


x/mobile/misc/androidstudio: GOMOBILE setting is not required

The gradle plugin should be able to find where the gomobile binary is with the GOPATH setting, the binary is under GOPATH/bin. We should make GOMOBILE an optional setting and use the binary under GOPATH/bin if GOMOBILE is not set.
cc/ @hyangah

That is already optional. Skipping GOMOBILE should work.
Let me know if it's not working.
2015-08-03 13:43 GMT-04:00 Burcu Dogan<EMAIL_ADDRESS>
The gradle plugin should be able to find where the gomobile binary is with
the GOPATH setting, the binary is under GOPATH/bin. We should make GOMOBILE
an optional setting and use the binary under GOPATH/bin if GOMOBILE is not
set.
cc/ @hyangah https://github.com/hyangah
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/11998.

--
__

I am modifying the reference bind example to make it clear.


Does AzureContainerAppsRC task support sending Environment Variables along with build Image?

Is it possible to send environment variables along with image (supported in 'az containerapp up' or 'az containerapp update' commands) using AzureContainerAppsRC task ?
I don't see environment variables in Arguments list provided here
https://marketplace.visualstudio.com/items?itemName=microsoft-oryx.AzureContainerAppsRC

@ranjeetkpGit Hey Ranjeet, apologies for the delayed response; @Basssiiie opened a PR in the azure-pipelines-tasks repository (for the stable AzureContainerApps task) that adds this functionality, and I'm looking to port this functionality over to this repository's AzureContainerAppsRC task in the near future along with some additional changes being made.
I'll keep this issue posted with updates as they are rolled out. Thanks!

@ranjeetkpGit Hey Ranjeet, this issue is resolved in the latest release of the AzureContainerAppsRC task (0.1.8) -- please feel free to let us know if there's anything else we can help with!
Thanks again for the contribution @Basssiiie :shipit:


Unrecognized font family 'FontAwesome'

I've build the application today using XCode and got the following error - it's a fresh clone:

MacOS Sierra: 10.12.2
XCode: 8.2.1 (8C1002)
npm: 3.10.9
node: v7.2.1
react-native-cli: 2.0.1
react-native: 0.37.0
I needed to run $react-native upgrade, due to the previous issue found here on the Issues list.
How can I solve this?

Actually this command helped me after $react-native upgrade:
react-native link react-native-vector-icons

Closing this as the issue seems to be resolved.


Fix Jason producing encoded JSON payloads without the apiKey field; make JSON library configurable

#84 introduced a bug. Since Jason cannot encode arbitrary structs, a custom Jason.Encoder implementation is required. If a Jason.Encoder implementation is derived, only known struct fields are going to be encoded (i.e. api_key, notifier, events). The Payload.new/3 constructor puts the apiKey field (which is used by the Bugsnag endpoint) into the struct and leaves api_key nil. Because of the derived implementation of Jason.Encoder, the apiKey is always ignored when the payload is encoded to JSON; it produces JSON objects like this:
{
  "api_url": null, # always null because `Payload.new/3` never sets this field
  "notifier": ...,
  "events": [...]
}

Because the Bugsnag endpoint never receives a valid API key, such reports are silently ignored. This PR fixes this bug.

I also made JSON library customizable (Jason is used by default).

@jarednorman ping.

@jarednorman can we please fix this? without this, this is basically dead in the water.

Thanks!

Apologies as this project is short on maintainers at the moment and I'm not doing any Elixir work these days. 🙂
I assume you'd like a new version cut.
@smaximov Would you call this a breaking change warranting a major version bump? At glance, I'd think so, but I don't have any time for testing this out just now.

@jarednorman, yes, I believe this is a breaking change because now both Jason and Poison are optional dependencies and the user is required to specify one of them. Otherwise, their app can break unless Jason is already installed as a transitive dependency of some other package.

Thanks, I'll bump the major version then. 🤘🏻

@jarednorman, can you please release a new version of Plugsnag? It currently depends on 1.x versions of Bugsnag which is incompatible with the new 2.0 release.


ARROW-10600: [Go] Implement Decimal256

I'm gonna bug you again @wolfeidau to have a look as we still lack many Go developers here who can take a look at these. :smile: After this, the union PR and one more PR afterwards (implementing unions in the IPC handling) Go will officially support all of the Arrow data types!

There seem to be a lot of Go CI failures suddenly.

@pitrou Yea, when I merged the Union type PR it added new requirements to the Builder types which I hadn't implemented in the Decimal256Builder so the build failed. I just had to add the new required interface methods to the Decimal256Builder and now all should be good.


module 'pkgutil' has no attribute 'ImpImporter'

i always end with this error (module 'pkgutil' has no attribute 'ImpImporter') when installing tafrigh !
and the script just pop up and disappear when i click on.
i am using Windows 11
and python 3.12

something about the ImpImporter class being removed/deprecated at some point: https://stackoverflow.com/questions/77364550/attributeerror-module-pkgutil-has-no-attribute-impimporter-did-you-mean
try this :
python -m ensurepip --upgrade
python -m pip install --upgrade setuptools

Tafrigh now is available through Docker with pre-built images, check the docs.