Add eslint for source code

I think @Globegitter has been pretty busy lately, so this is an offer to implement just the linting part of https://github.com/balderdashy/waterline/pull/935.  I am willing to go through and try to fix failing rules, but I would like to get consensus on what the rules should be prior to spending the effort.  With the rules from https://github.com/balderdashy/waterline/pull/935 (currently in this PR), I get the following results:

(Note that all these screenshots are cropped on the right, a few bars are much longer)
Another strategy to avoid bikeshedding and arguing over rules, would be to use eslint-config-standard. With that set of rules, these are the results:

My suggestion would be to override the following rules to more closely align with the existing codebase:
"rules": {
  "space-before-function-paren":  [2, "never"],
  "space-after-keywords":         [2, "never"],
  "semi":                         [2, "always"]
}

The results of these three changes results in something pretty manageable:

What say ye?

I've made sure that all linting rules are warnings, such that this could be merged at any time and will not break the tests.  I still would recommend agreeing on the rules before going forward with updating the code.

:+1:
I'm not too familiar with eslint rules, I've been using jshint and pretty much stick with the defaults. @IanVS do you have any opinions about which rules we should change in regards to the defaults?

ESLint has pretty minimal "default" rules, and even those will be going away in the upcoming 1.0 release.  It bills itself as unopinionated.  The "JavaScript Standard Style" is a pretty good set of rules I think.  Perhaps what I can do is make a sample commit for each of the rules, with one change, so you can see the diff and decide if you like it.  I'm not too familiar with the waterline code base, so it's hard for me to say which rules make the most sense for you.

The commits named "Fix ESLint Rule" are the easy ones (mechanical, not much thought needed, not an overwhelming number of them).
For the rest of the changes, I'll do a few examples as a "preview" and ask for guidance before continuing.

I'm good with these changes.
But would like hear other people's opinions such as @devinivy, @tjwebb and @particlebanana.

I think it's great :+1:

Would you like me to squash these commits before proceeding to the slightly-more-difficult changes?  Or, I'm happy to leave them separate, which I think makes it very clear to see what was done for each rule (both now and in the future).

Please take a look at the commit messages labeled "Example" and give your feedback on whether to proceed with fixing the code as I started, or whether we should tweak the rules more to your liking.
There are a handful of other rules I did not attempt to fix.  These are:
9 errors: new-cap: [1, { "newIsCap": true, "capIsNew": false }]
7 errors: no-proto: 1
13 errors: eqeqeq: [1, "allow-null"]
3 errors: no-new: 1
I think these are all pretty reasonable, and they can be left in a warning state until the Waterline team can examine and fix them, or turn the rules off if they can't be fixed.  I just don't have the knowledge of your code needed to fix those four rules.

@IanVS, the PR is good for now. Lets see if we can get some feedback from @particlebanana or @tjwebb.

Hopefully there can be a review soon, so I don't need to rebase with master and then make even more changes to fix new linting errors.  :wink:

@IanVS I'm looking over this today, not familiar with ESLint so I'm looking over it and all the rules we have here.

@particlebanana Awesome, thanks. It can take a little time to research the rules, but I've found the docs to be quite good. The configuration I'm proposing here mostly follows https://github.com/feross/eslint-config-standard/blob/master/eslintrc.json, with a few exceptions as mentioned. If you have any questions, let me know and I'll be happy to try to answer.

This is awesome! Thanks @IanVS and I'm ashamed of some of the stuff in there!

Haha, lets work on getting a nice set of rules that make sense to us, fix all issues and then make rules return errors when broken! :)
Thanks @IanVS!

Great, glad this is useful.  Here are what I see as the decisions to be made:

no-multiple-empty-lines

[ ] 1 empty line
[ ] 2 empty lines

space-after-keywords

[ ] Always
[ ] Never

space-before-function-paren

[ ] Always
[ ] Never
[ ] Always for anonymous, never for named

For now I can work on the other rules like one-var, no-unused-vars, and brace-style.   Once decisions are reached on the above, I can work on those too.  The following will need to be addressed by the Waterline team:

no-undef
new-cap
no-proto
eqeqeq
no-new

Now that this is merged, linting will occur on each test run, but it will look a little overwhelming.  You can also use npm run nibble to see rule summaries and then focus in on a specific error.

no-multiple-empty-lines

[ ] 1 empty line
[x] 2 empty lines

Given how abundantly empty lines are used in waterline I vote for allowing 2 lines (this will still allow just one line, right?).

Yes, it just means max of two sequential empty lines.

I vote,

no-multiple-empty-lines

[ ] 1 empty line
[x] 2 empty lines

space-after-keywords

[x] Always
[ ] Never

space-before-function-paren

[ ] Always
[x] Never
[ ] Always for anonymous, never for named

I vote:
no-multiple-empty-lines: 2
space-after-keywords: always
space-before-function-paren: never
no-undef: true
new-cap: true
no-proto: true
eqeqeq: true - smart
no-new: true

I vote for one max empty line. We can split the different and make it 1.5? :)

I'm afraid your math is off:
@dmarcelino : 2
@devinivy: 2
@particlebanana: 2
@tjwebb: 1
Average: 1.75 :stuck_out_tongue:


Check actual headers

What kind of change does this PR introduce? (check at least one)
Fix a bug that not adding unprotected headers to signatures.

[x] Bugfix

Does this PR introduce a breaking change? (check one)

[x] No

[x] You consent that the copyright of your pull request source code belongs to Authlib's author.

Thanks, but you got it wrong. The original implementation is right, but the logic is not well to understand, I will change it.

Thanks. I'm doing a lot of refactoring right now, because of https://github.com/lepture/authlib/issues/65 I will take it into the refactoring.

It is fixed in the feature-jwe branch.

Thanks for addressing that!


[Bug]: Editable option is missing on binding the table widget with some data

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Editable option is missing on binding the table widget with some data.
Steps To Reproduce

D&D a table widget onto the canvas
Bind with some data
Notice that the editable checkbox isn’t available
Expected:

Actual:

Public Sample App
No response
Environment
Production
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
v1.32

Closing this since its an intended change


What about Soundcloud?

Greetings

Soundcloud and Deezer (french website) would be great. Maybe I fill find some time to make a PR for thoses.


History file

Hello!
The history file didn't work on my MBP so I checked the script and made 2 tweaks :)

Thank you very much! :)


Null Object reference with thorns bgGhouls

Describe your bug here.
When loading into thorns it will give out a error from playState line 2829 with this code presumably (i highlighted the code)

Command Prompt/Terminal logs (if existing)
PlayState.hx (line 2829)
flixel/FlxState.hx (line 203)
flixel.FlxGame.hx (line 752)
flixel.FlxGame.hx (line 682)
flixel.FlxGame.hx (line 550)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)

Are you modding a build from source or with Lua?
Source
What is your build target?
Linux
Did you edit anything in this build? If so, mention or summarize your changes.
I added a SkinPicker (its in the works but I was testing it and found this error)

interesting enough, i have the same problem in another moded psych engine

interesting enough, i have the same problem in another moded psych engine

Also i enabling low quality works, but i see that the characters aren't in the right places

same error for me when loading MILF


mapToModel fixes

ViewModel.mapToModel would break down when the view model was composed of multiple aliased joins on the same table. This PR adds a new version of the mapToModel method that takes an aliased table argument to correct this problem. The single-arg version will still work as expected if the view model joins on a table only once, but will log an error and return null if it detects multiple aliases for the same class are present.

I'd like to also address #152 with this PR, so let's hold off on merging it until I can get a handle on that as well.

I've added the changes that fix #152, so this is now ready for a good thorough review.

LGTM


Add a dev page for testing logos

It can be helpful to have some diagnostic pages for development and quality control. I added one here for the logos, which renders all the named logos in ?style=flat and ?style=social.

Yes - there are definitely some light logo on light background issues with our bespoke logo set
It would be quite nice to make this view (and potentially other 'debug' views we might want to declare) available in dev, but not in production. I've slightly lost track of this in the settings re-organisation, but at the point we build the front-end, do we have an env var available that indicates prod/dev? i.e: can we declare the route conditionally on proceess.env.ENVIRONMENT == 'dev' or something so it exists in local dev, but not when we do the optimised production build?

i.e: can we declare the route conditionally on proceess.env.ENVIRONMENT == 'dev' or something so it exists in local dev, but not when we do the optimised production build?

That sounds good. Let's get #2906 merged and then revisit this.

We still may need to solve the rate limiting, though even with it, I think this page remains more useful than not.


Update has broken Zed's default executables

Describe the bug
Zed's default formatter Prettier no longer runs for me.
I've tracked it down to Mise activation in ~/.zshrc.
When mise is activated and I open a .tsx file in Zed, it starts up multiple mise processes, the fans start spinning and the formatters time out.
To Reproduce
Activate Mise in ~/.zshrc
eval "$(/opt/homebrew/bin/brew shellenv)"
eval "$(mise activate zsh --shims)"

Download and install the Zed text editor:  https://zed.dev/download
Launch Zed and make a new text file test.tsx
Enter function test() {alert("a")} and save file

Expected behavior
Zed's default settings for .tsx files are to format with Prettier.
function test() {
  alert("a");
}

Prettier is installed to ~/Library/Application Support/Zed/prettier.
It's not clear to me why these are triggering these mise processes or what is hanging.
Can you shed any light on this?

I'm not going to debug it for you but you can setup file logging as described here to see what mise is doing

It's related to shims somehow.  This works as expected.
- eval "$(mise activate zsh)"
+ eval "$(mise activate zsh --shims)"

I've had one issue after another with shims the last two weeks, will avoid it.

while I don't have logs to confirm I think https://github.com/jdx/mise/commit/35d31a1baf96fe6f0e764e26228c1b03ba24ddce may have fixed this


New version: JetBrains.GoLand.EarlyAccess version 231.6471.12

Result: Installation Failed

Manifest
Add/Remove Programs

Name
GoLand (EAP)

Version
231.6471.12

Publisher
JetBrains s.r.o.

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 5751
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


How to open task-web to remote local machines in the same network?

Hi there,
Great tool, I have been using a lot in my desktop. However, I'd like to reach to task-web from my other LAN machines.
For example I run task-web on http://<IP_ADDRESS>:5678/
But I can not open the same address on the same network. I tried running task-web with different flags.
Do you know how I can do that?
Thanks.

Which exactly command do you use to run task-web?

I tried different flag combos of task-web however it didn't make much difference.
This is what i use currently:
task-web --host <IP_ADDRESS> --env production  -F -d

I'm not able to test it in LAN, but I'm sure you need to use <IP_ADDRESS>. In this case task-web will accept connections from any IPs

No luck there. I have retried that doesn't work. Ultimate goal is remotely seeing task-web from outside network via port forwarding. Fw rules are set, my python servers are just able to work remotely on WAN and LAN but task-web is not. Feel like it's a webrick issue maybe?

I did it in the past. Task-web was running on a laptop and was accessible from phone in the same LAN. I'll try it tonight.

Thanks looking forward to it!

any update here?

Sorry, I was overwhelmed last days.
So, for me it works.
0. Didn't have taskwarrior-web installed, so run gem install taskwarrior-web.

On my laptop I ran task-web without any additional flags.
Find my laptop's IP in LAN: it is <IP_ADDRESS>.
On my phone go to <IP_ADDRESS>:5678.
Done

Maybe some firewall restrictions on your host machine?

I'm on Mac by the way, firewall is disabled in system settings.

Thanks for confirming that it works out of box.
This did the trick. I'm using ubuntu subsystem in windows.
https://www.nextofwindows.com/allow-server-running-inside-wsl-to-be-accessible-outside-windows-10-host
Now I'm able to see my tasks, yay!


Pin Diagram for connecting ESP8266 and DHT with Arduino Uno

How is ESP8266 and DHT are connected with Arduino Uno ?
What is the connection diagram for that ?

Not sure if this directly related to this library.
You have example on how to hook an ESP-01 to a DHT here:
https://learn.adafruit.com/esp8266-temperature-slash-humidity-webserver/wiring


Store has already been added app

Currently I have 2 stores, AppStore and AuthStore. AppStore is controlled by the AppModule for general all-purpose states for the app. AuthStore is managed by the AuthModule.
In AppModule:
  imports: [
    //...
    NgxsModule.forRoot([AppStore]),
    AuthModule,
    // ...
  ]

In AuthModule:
  imports: [
    //...
    NgxsModule.forFeature([AuthStore]),
    AuthModule,
    // ...
  ]

If I do this in AuthModule, I get an error:

Error: Store has already been added app
at _loop_1 (ngxs-api.js:77)
at StoreFactory.add (ngxs-api.js:98)
at StoreFactory.addAndReturnDefaults (ngxs-api.js:114)
at NgxsFeatureModule.initStores (ngxs-api.js:455)
at new NgxsFeatureModule (ngxs-api.js:451)
at _createClass (core.js:10895)
at createProviderInstance$1 (core.js:10865)
at initNgModule (core.js:10818)
at new NgModuleRef (core.js:12073)
at createNgModuleRef (core.js:12062)

AuthModule isn't LazyLoaded. If I removed the import from AuthModule and add the AuthStore to the import in AppModule it works. I'd rather not add the AuthStore into the AppModule if I can help it.
Am I not doing this correctly? Does forFeature() only work if the module is lazy loaded?

If I change AuthModule to use forRoot() instead of forFeature() it works. Not at all what I was expecting. Usually forRoot() means it should only be called once from a root module.

forFeature is only for lazy loaded routes. You should import it in the forRoot in this case.

@amcdnl I am not sure about this. Ngrx had the same problem, lots of people nagged, they finally allowed empty forRoot([]) when they move to ngrx/platform.
Not all features are meant to be lazyLoaded. A self-contained module should be able to take care of its own store and everything that goes with it.
For example, think of an Auth module. Just because it is put it it's own module in a libs/ directory, doesn't mean that it has to be lazyLoaded. Here app.module.ts imports auth authModule doesn't have to worry about anything else.
I wonder if there are any restrictions that prevent forRoot([]) implementation?

@un33k - I'm down for trying that. @deebloo - this sounds like up your alley.

@yarrgh I think this may have been fixed by @leon work with the feature modules. can you check again with the latest rc?

With 2.0.0-rc.18 I get a different error.

'NgxsModule.forRoot()' was not called at the root module

I do have NgxsModule.forRoot([]), in my app.module.ts and immediately after I'm importing my AuthModule that imports NgxsModule.forFeature([AuthState]). If I remove the forFeature() import form my AuthModule it works (except for the AuthState not being set up, obviously).

Here's a StackBlitz demonstrating the error:
https://stackblitz.com/edit/ngxs-simple-amvqoh

@yarrgh the problen is with the feature. if you remove AuthModule from :
@NgModule({
  imports:      [
    BrowserModule,
    NgxsModule.forRoot([CountState]),
    AuthModule ],
  declarations: [ AppComponent ],
  bootstrap:    [ AppComponent ]
})

works correctly.
The problem is:
@NgModule({
  imports: [
    CommonModule,
    NgxsModule.forFeature([AuthState]),
  ],
  declarations: [],
  exports: [],
  providers: []
})

You use AuthState with forFeature function, but AuthModule  I not being loaded lazy loading

From @un33k's comment:

Not all features are meant to be lazyLoaded. A self-contained module should be able to take care of its own store and everything that goes with it.

I think forFeature() should work for non lazy loaded modules as well as lazy loaded modules. Having AuthModule be lazy loaded doesn't make much sense since it is always needed in every aspect of my app.
Maybe if it is easier have a forFeature() for non lazy loaded modules and a forLazyFeature() for lazy loaded modules?

@yarrgh I agree with you! forFeature()  should work for non lazy loaded modules as well as lazy loaded modules.
it seems that's the bug

Closing this issue now that the fix has been released


Updating description to reflect article purpose

..as the article doesn't include information on the failover vNet networking , etc.

#sign-off


Remove primitive tangents and refactor device constructor

Remove pc.precalculatedTangents which is not used.
Add option to calculate tangents for primitives but disable it by default.
Destroy framebuffer after renderable float texture test.
Remove closure for graphics device constructor.
Fix some bugs related to setting parameters in the standard material.

I confirm I have signed the Contributor License Agreement.

Cool. So it seems to only affect primitives for now? For meshes I guess we need some kind of a checkbox on assets.

Yep, exactly! I'll have a chat with @vkalpias and @daredevildave to work out the details.


One site not getting v 4.9.1

[x] This is a question about using the theme.
[ ] This is a feature request or enhancement.
[ ] I believe this to be a bug with the theme.

[x] I am using the latest version of the theme.
[x] I have updated all gems with bundle update.
[x] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version: 4.9.0/4.9.1
Jekyll version: 3.7.2
GitHub Pages hosted: no
Operating system: Windows 10

I'm trying to figure out why one of my sites doesn't seem to be picking up the 4.9.1 version of the theme, while I have another one configured the same way that is getting the new version. Both are set up as the gem-based theme, not hosted on GitHub pages. I've run bundle update and it says Using minimal-mistakes-jekyll 4.9.1, but when I view the source of the generated page, I see Minimal Mistakes Jekyll Theme 4.9.0 by Michael Rose. This happens both locally and when Netlify builds the site. Any ideas what I can be missing or what else I should check?

If you Gemfile.lock says 4.9.1 then you have the latest version installed.
Other things to check are to make sure you've flushed your browser's cache. This usually affects the .css and .js more so than the .html files.
Also, do you have any _layouts you are overriding? I got bit by this once where I created a modified version of _layouts/default.html which replaced the bundled theme's version. This file is what prints the theme's version number.

That's it! I was overriding _layouts/default.html on that site. Duh. Thank you!

:smile: No worries.


Add 'x' on Contact Us and Learn More

It currently closes by clicking outside of the phone. That should work, but we should also have an 'x' for easiness.

See #12


Please support Markdown within HTML tags.

On Version 1.6.0 Markdown within HTML tags like <p>[[this]]</p> appears not to work for me:
```chat
< me | <p>This [[link]] is **not** linked.</p>
> also me | This [[link]] works.<br />It's a **workaround**.
```

Is it possible to support Markdown within enclosing HTML tags?
Thank you for this awesome plugin!

Hi @hinnerk. I've looked into this. Sorry, for the delayed reply. Turns out, if there are HTML Tags involved in the text, the official Obsidian Markdown Renderer interprets it as HTML code and doesn't actually render the markdown syntax.
I'll see if I can find a workaround. Will get back to you soon.


parser error

hello i am getting this error, and whenever i restart my machine i have to reinstall my king phisher,
please tell how to install plugins, and how to make first campaign.
it is not necessary to run the king phisher client as root
ERROR    css parser error (not a number) in /root/Desktop/king-phisher/data/client/king_phisher/style/theme.v2.css:98
ERROR    css parser error (Expected a string.) in /root/Desktop/king-phisher/data/client/king_phisher/style/theme.v2.css:98
ERROR    there was an error parsing the css file, it will not be applied as a style provider

The theme CSS file appears to be corrupted, you should restore it from the repository. In the future please fill out all of the fields in the issue template, specifically you haven't provided details regarding what system you are running this on.
For help with logging in please see the tutorial videos available in the wiki. Additionally steps for installing plugins is available in the wiki.

ok buddy, now its showing connection refused,
i am running kali aka, with updated version of king phisher

Are both the King Phisher service and the SSH services running?

Is ssh setup on your kali machine? Are you using your kali machine for both the server and client?
Sent from my T-Mobile 4G LTE Device
-------- Original message --------
From: sealnil<EMAIL_ADDRESS>Date: 9/24/16 10:48 (GMT-05:00)
To: securestate/king-phisher<EMAIL_ADDRESS>Subject: Re: [securestate/king-phisher] parser error (#154)
ok buddy, now its showing connection refused,
i am running kali aka, with updated version of king phisher

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/securestate/king-phisher/issues/154#issuecomment-249368405, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHYH4R7o-NMoeF39NnJf7S3cluUFzlfcks5qtTgbgaJpZM4KFqCB.

yes i have ssh set up on machine along with king phisher client, not config server,
the commands i used
apt-get install openssh-server
after downloading king phiser , extract it on desktop, open in terminal
cd king-phisher
cd tools
sudo bash ./install.sh
and enter uname as root and machine password to loging, earlier i was logged in and showing new campaign, and other options, after after today i started up the machine it is connection refused error.
this much i have done till now,
please tell where i am doing wrong or have i missed any step in between.
thanks in advacne

It sounds like you need to configure and start the server. Also please copy and paste the exact connection error message into the issue. If the server is started, please share the logs with us as well. We'll need all that information to help.

King-Phisher Server Configuration

logging:
file: /var/log/king-phisher.log
Available levels: FATAL, ERROR, WARNING, INFO, DEBUG
level: warning
console: false
server:
Bind address information, multiple ports can be
addresses:
- host: <IP_ADDRESS>
port: 80
ssl: false
Server RPC authentication related settings
#authentication:
# How long to cache authentication information in the server process
#cache_timeout: 10m
# Require authenticating users to be a member of the specified group
#group: king-phisher
# The service to use for PAM authentication
#pam_service: sshd
# How long an authenticated RPC session should be valid for after no
# requests have been received.
#session_timeout: 30m
A cookie to retrieve the client's IP from, useful when a proxy is in place
#client_ip_cookie: X-Real-IP
The name of the cookie used to track users
cookie_name: KPID
Specify an additional search path for server data files
data_path: /opt/king-phisher
The SQLite3 database file to use, the folder needs write permissions
WARNING: SQLite3 is not fully supported as of version 0.1.6, see
https://github.com/securestate/king-phisher/wiki/Database#sqlite for more
details
#database: sqlite:////var/king-phisher/king-phisher.db
The PostgreSQL database to use
database: postgresql://king_phisher:6MWSbKiYa6FjBAuxuZ3Q96eXS6S1aAJ0pxEOrUH6@localhost/king_phisher
Fork the server process into the background
fork: true
geoip:
# The location of the GeoLite2 City database
database: /var/king-phisher/geoip.mmdb
Set a file to write the PID to
pid_file: /var/run/king-phisher.pid
Add additional search paths for plugin directories
#plugin_directories:
- /some/plugin/path
List the plugins that need to be enabled
#plugins:
# The plugin to enable's name
some_plugin:
# An option specific to this particular plugin
value_for_some_plugin: true
Require an ID associated with a campaign to load pages
require_id: true
rest_api:
enabled: false
# Networks which are authorized to use the REST API
networks:
- <IP_ADDRESS>/8
- <IP_ADDRESS>/8
- <IP_ADDRESS>/12
- <IP_ADDRESS>/16
# When null the token will be randomized at run time
token: null
Configure the server header value
#server_header: Apache/2.4.9 (Unix)
Set the opened timestamp on the first visit if it's null
set_message_opened_on_visit: true
Drop privileges to the specified user
setuid_username: nobody
Default SSL certificate settings
#ssl_cert: /path/to/ssl.crt
#ssl_key: /path/to/ssl.key
Hostname specific SSL certificates can be added below
#ssl_hosts:
- host: www.king-phisher.com
ssl_cert: /path/to/www.king-phisher.com/ssl.crt
ssl_key: /path/to/www.king-phisher.com/ssl.key
Configure the image file name used to track opened emails
tracking_image: email_logo_banner.gif
Use subfolders in the web_root which correspond to vhost names
vhost_directories: false
this is the config file of server
and now i am able to loging as connection refused error is poping

root@kali:~# service ssh status
● ssh.service - OpenBSD Secure Shell server
Loaded: loaded (/lib/systemd/system/ssh.service; disabled; vendor preset: dis
Active: active (running) since Sat 2016-09-24 19:47:47 IST; 3h 50min ago
Main PID: 1719 (sshd)
Tasks: 1 (limit: 4915)
CGroup: /system.slice/ssh.service
└─1719 /usr/sbin/sshd -D
Sep 24 19:47:47 kali systemd[1]: Starting OpenBSD Secure Shell server...
Sep 24 19:47:47 kali sshd[1719]: Server listening on <IP_ADDRESS> port 22.
Sep 24 19:47:47 kali sshd[1719]: Server listening on :: port 22.
Sep 24 19:47:47 kali systemd[1]: Started OpenBSD Secure Shell server.
lines 1-12/12 (END)
ssh server is active and running

What is the status of your King Phisher service? Please don't post your confirmation file again as it contains your Postgresql connection information including the password.

ok thanks i will take care next time
still connection refused, should i remove and reinstall it?

No you need to make sure that the king-phisher systemd service is running with systemctl.

plz send the command format for checking king-phisher systemctl

King Phisher Server
Loaded: loaded (/lib/systemd/system/king-phisher.service; enabled; vendor preset: disabled)
Active: failed (Result: resources) since Sat 2016-09-24 23:30:38 IST; 24min ago
Process: 19984 ExecStart=/usr/bin/python3 /opt/king-phisher/KingPhisherServer /opt/king-phisher/server_config.
Sep 24 23:30:37 kali python3[19984]:   File "/usr/local/lib/python3.5/dist-packages/sqlalchemy/engine/default.py
Sep 24 23:30:37 kali python3[19984]:     return self.dbapi.connect(*cargs, **cparams)
Sep 24 23:30:37 kali python3[19984]:   File "/usr/local/lib/python3.5/dist-packages/psycopg2/init.py", line
Sep 24 23:30:37 kali python3[19984]:     conn = _connect(dsn, connection_factory=connection_factory, async=async
Sep 24 23:30:37 kali python3[19984]: sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) FATAL:  passwo
Sep 24 23:30:37 kali python3[19984]: FATAL:  password authentication failed for user "king_phisher"
Sep 24 23:30:38 kali systemd[1]: king-phisher.service: Daemon never wrote its PID file. Failing.
Sep 24 23:30:38 kali systemd[1]: Failed to start King Phisher Server.
Sep 24 23:30:38 kali systemd[1]: king-phisher.service: Unit entered failed state.
Sep 24 23:30:38 kali systemd[1]: king-phisher.service: Failed with result 'resources'.
~

It's sudo systemctl status king-phisher
In the future please try to find the answer to these basic questions yourself using a search engine like Google.

sure

sudo systemctl status king-phisher
● king-phisher.service - King Phisher Server
Loaded: loaded (/lib/systemd/system/king-phisher.service; enabled; vendor preset: disabled)
Active: failed (Result: resources) since Sat 2016-09-24 23:30:38 IST; 28min ago
Process: 19984 ExecStart=/usr/bin/python3 /opt/king-phisher/KingPhisherServer /opt/king-phisher/server_config.
Sep 24 23:30:37 kali python3[19984]:   File "/usr/local/lib/python3.5/dist-packages/sqlalchemy/engine/default.py
Sep 24 23:30:37 kali python3[19984]:     return self.dbapi.connect(*cargs, **cparams)
Sep 24 23:30:37 kali python3[19984]:   File "/usr/local/lib/python3.5/dist-packages/psycopg2/init.py", line
Sep 24 23:30:37 kali python3[19984]:     conn = _connect(dsn, connection_factory=connection_factory, async=async
Sep 24 23:30:37 kali python3[19984]: sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) FATAL:  passwo
Sep 24 23:30:37 kali python3[19984]: FATAL:  password authentication failed for user "king_phisher"
Sep 24 23:30:38 kali systemd[1]: king-phisher.service: Daemon never wrote its PID file. Failing.
Sep 24 23:30:38 kali systemd[1]: Failed to start King Phisher Server.
Sep 24 23:30:38 kali systemd[1]: king-phisher.service: Unit entered failed state.
Sep 24 23:30:38 kali systemd[1]: king-phisher.service: Failed with result 'resources'.

The service is not running because the password for the database is incorrect. There is a section in the FAQ page of the wiki on how to address this.

I'm going to close this issue since the solution is in the FAQ and there hasn't been any response.


[suggestion] Support for Android

I've been a user of BetterDiscord for a very long time, and I wanna see support for Android (At least to the point of Twitch emotes
-whamer100

Frankly, that's not really possible. It would require either rebuilding the whole Discord app or using Xposed. For both, the knowledge of either Java or C# and Android itself is required.
Additional to that, the users that want it would have to install the Xposed Framework on their phones.
So, no. There won't be Android support anytime soon

Frankly Its not even as easy as what you said. I looked into the app code some time ago to do some custom styling myself and I failed to overwrite a single color. Apart from that, I would consider myself as knowledge of Java and Android.
Besides, no C# in android apps.

Well, now we have tools like apktool and magisk. If com.discord isn't proguarded I'll take a look at my possibilities.

p8tgames betterdiscord would be really cool for android.

Any Update on this p8tgames?

Imma try make this a reality 2morrow

Any breakthroughs?

Anything?👀

Hello!
Sadly no. I did take multiple looks at the discord app and found ways to
enable some hidden gems before, but I’m not very experienced with smali and
other required stuff.
Lost0Neko<EMAIL_ADDRESS>schrieb am Sa. 30. Mai 2020 um 20:58:

Anything?👀
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Jiiks/BetterDiscordApp/issues/294#issuecomment-636370862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBMA5K36BFHGZDLE46X6WLRUFJNFANCNFSM4CC2YKFQ
.

Ok, I found a way to do this but will take lots of time and only themes will be able to be implemented. You decompile the app with apktool, look for stuff in the xml that you can replace and make a xposed module.

you might as well make a substratum theme then
On Fri, 24 Jul 2020 at 13:42, xSlendiX<EMAIL_ADDRESS>wrote:

Ok, I found a way to do this but will take lots of time and only themes
will be able to be implemented. You decompile the app with apktool, look
for stuff in the xml that you can replace and make a xposed module.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Jiiks/BetterDiscordApp/issues/294#issuecomment-663498774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBMA5NZ5LRYGQXX3UQL373R5FXRLANCNFSM4CC2YKFQ
.

Its 2020 anything?

Bluecord

Thx

I wanna see where this goes now

Bluecord

https://www.virustotal.com/gui/file/b136e544c1a8b9ee0e9699332c536a432a80c5c437598b1d2d478df58757b12b/detection


eslint v9 compatibility

The latest version of the plugin (11.0.3) is not compatible with eslint v9. Trying to use it results in the error TypeError: context.getScope is not a function.
According to the official eslint docs, this happens because

ESLint v9.0.0 introduces changes to the rules API that plugin rules use, which included moving some methods from the context object to the sourceCode object

What about creating a new major version that only supports eslint v9? Or is it your intention to continue to make changes to the plugin in the future, and you explicitly want to make those changes available to older eslint versions?

I am not currently investing effort in this plugin, so fixing breaking changes that eslint decided to make (which I still sorta disagree with, but whatever) is not high on my list. But moreover, having then to need two separate projects (or branches or whatever) that have to be maintained --not just adding of features, but fixing bugs or security issues, etc -- is even more work, and even harder to justify.
I wouldn't block efforts by other contributors to do so, but it's just not something I have planned to prioritize here.
So, IOW... PRs accepted. ;-)

I have opened a PR https://github.com/getify/eslint-plugin-proper-arrows/pulls
in case Kyle is willing to consider it

thanks @Smrtnyk for all the contributions to land those changes!


Add footer (or other thing) with links to about, rules, legal, etc

At the moment this is covered by the "Whoa! What's going on here?" dialog but that feels like more of a temporary solution.
This would also let us get around to addressing #508 :P

In the process of putting something simple together. I'm thinking something in the sidebar would work. Not sure about the copy yet

We'd also need to add a legal page, which looks like is handled through the database.


GRA-1205: internet gateways

Describe your changes
Internet gateways on linux only work with this PR
Will merge after Mac and Windows complete initial PRs
Provide Issue ticket number if applicable/not in title
title.
Provide link to Netmaker PR if required

https://github.com/gravitl/netmaker/pull/2182

Provide testing steps

[ ] Ensure that an egress gateway with <IP_ADDRESS>/0 functions as an internet gateway for a connected linux peer

Checklist before requesting a review

[ ] My changes affect only 10 files or less.
[x] I have performed a self-review of my code and tested it.
[x] If it is a new feature, I have added thorough tests, my code is <= 1450 lines.
[ ] If it is a bugfix, my code is <= 200 lines.
[ ] My functions are <= 80 lines.
[x] I have had my code reviewed by a peer.
[x] My unit tests pass locally.
[x] Netclient & Netmaker are awesome.

In order to compile on Windows, should have unimplemented functions in routes_windows (see workflow failures)

I state in PR won't be merged till other ones are in...

I still have nodes freezing up when I set an egress gateway to <IP_ADDRESS>/0


Avoid NPE when match list is empty

Hey, thanks for the PR. Would you mind also applying this change to all other appropriate overloadings of "getMatchList" before I merge it?

Sure, I won't have access to a computer until Sunday but I can fix it after
that.
On Wed, Aug 17, 2016, 16:28 Rob Rua<EMAIL_ADDRESS>wrote:

Hey, thanks for the PR. Would you mind also applying this change to all
other appropriate overloadings of "getMatchList" before I merge it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meraki-analytics/Orianna/pull/56#issuecomment-240429509,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGIlJI5t-9YNSoqlLTzGAAqJaXERZoS4ks5qgxqcgaJpZM4Jgose
.

There we go. @robrua

Hey - sorry for being so slow on this. I'd like to merge in 807f966 if you wouldn't mind rolling it back to that - the DTO objects are supposed to reflect the exact response from the API with no usability changes, so I don't want to return empty lists when nothing was sent.

Okay yeah, that makes sense. Reversing the last commit now.

Thanks for the PR!

You're welcome. :)


Improve telemetry around Wasm extensions

Tracking bug: need more telemetry around Wasm extension behavior. We need to be able to monitor what extensions are deployed, how the VMs are behaving, etc.
[ X ] Policies and Telemetry

not stale


The tracing chain is not complete when Spring Cloud Zuul enable okhttp3

Question
The tracing chain is not complete when Spring Cloud Zuul enable okhttp3.

spring-boot.version: 2.0.3.RELEASE
spring-cloud.version: Finchley.RELEASE

In our test scenario, we call the test-demo-api through spring cloud zuul and want to get the tracing data from zipkin. Here is the zipkin configuration:
spring:
  application:
    name: ccf-gateway-app
  zipkin:
    enabled: true
    sender:
      type: web
    base-url: http://xxx.xxx.xxx.xxx:9411
  sleuth:
    sampler:
      probability: 1.0

Here is the tracing chain when we use apache http client:
application.yaml
--------
ribbon:
  ReadTimeout: 2000
  ConnectTimeout: 1000
  httpclient:
    enabled: true
  okhttp:
    enabled: false

Here is the tracing chain when we use okhttp3:
pom.xml
---
<dependency>
    <groupId>com.squareup.okhttp3</groupId>
    <artifactId>okhttp</artifactId>
</dependency>
<dependency>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-netflix-zuul</artifactId>
</dependency>
<dependency>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-zipkin</artifactId>
</dependency>

application.yaml
--------
ribbon:
  ReadTimeout: 2000
  ConnectTimeout: 1000
  httpclient:
    enabled: false
  okhttp:
    enabled: true

As you see, the tracing data miss the test-demo-api part when we enable okhttp3.
We found this link brave-instrumentation-okhttp3, is this a bug in spring-cloud-starter-zipkin?

i have same issue

import
    <dependency>
        <groupId>io.zipkin.brave</groupId>
        <artifactId>brave-instrumentation-okhttp3</artifactId>
        <version>5.6.0</version>
    </dependency>

add source :
@Bean
public OkHttpClient.Builder okhttpBuilder(Tracing trace) {
return new OkHttpClient.Builder().addNetworkInterceptor(TracingInterceptor.create(trace));
}


Show picker on input focus together with colorpicker-input-addon

Description
Is there a built-in way to force colorpicker to open on text input focus and have the colorpicker-input-addon button? looked in the docs but couldn't find anywhere.
Context

bootstrap-colorpicker version: 3.2.0
bootstrap version: 4
jQuery version: 3.2

Expected behavior
When color picker input is in focus, colorpicker component opens even when you use the colorpicker-input-addon.
Actual behavior
colorpicker opens only when pressing on colorpicker-input-addon
Thanks!

This is builtin functionality since the library automatically detects the addon
Did you try using events and opening the picker on focus?


Resize union before adding tags that would otherwise overflow its size

Also, this PR should package an iterator over equivalence classes: I give an hash-consed element and it gives me all the hash-consed elements in its equivalence class


Update behavioranalytics.md

Test should have the full title of the product within it not just "Sentinel"

please review and approve

@rboucher
I think the proposed change updates an auto-gen file. Can you submit a fix for the source? Or can the change be merged in this PR?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @BillClarksonAntill
As per your previous contribution, the product is now called Microsoft Defender for Cloud.
I have forwarded this to the developers to update the manifest file with the log descriptions.
#please-close


Per Monitor Layout Options

I have a two monitor setup, both 1080p. One is vertical while the other one is horizontal. One feature that I can't seem to find any way to do is to have my master layout orientation be set to Left/Right on my horizontal monitor and have it also be top/bottom on my vertical monitor. I prefer the master layout but I'll probably have to switch to dwindle until/unless this gets implemented.

dupe of #622


FEEL expressions with long strings are marked as invalid - even though they work

Describe the bug
FEEL expressions with long strings are marked as invalid - even though they work (can be deployed to C8, run on C8).

Steps to reproduce

Import the following BPMN
See the Script task implementation
Notice that error is indicated

<?xml version="1.0" encoding="UTF-8"?>
<bpmn:definitions xmlns:bpmn="http://www.omg.org/spec/BPMN/20100524/MODEL" xmlns:bpmndi="http://www.omg.org/spec/BPMN/20100524/DI" xmlns:dc="http://www.omg.org/spec/DD/20100524/DC" xmlns:zeebe="http://camunda.org/schema/zeebe/1.0" xmlns:di="http://www.omg.org/spec/DD/20100524/DI" xmlns:modeler="http://camunda.org/schema/modeler/1.0" id="Definitions_1ctgddk" targetNamespace="http://bpmn.io/schema/bpmn" exporter="Camunda Modeler" exporterVersion="5.15.0" modeler:executionPlatform="Camunda Cloud" modeler:executionPlatformVersion="8.2.0">
  <bpmn:process id="Process_0t4d2bh" isExecutable="true">
    <bpmn:startEvent id="StartEvent_1">
      <bpmn:outgoing>Flow_19rv9cw</bpmn:outgoing>
    </bpmn:startEvent>
    <bpmn:sequenceFlow id="Flow_19rv9cw" sourceRef="StartEvent_1" targetRef="Activity_1m1fzrs" />
    <bpmn:scriptTask id="Activity_1m1fzrs">
      <bpmn:extensionElements>
        <zeebe:script expression="=replace(replace("### Roles this week

| Role | DRI |
|---|---|
| Moderator | @---moderator--- |
| Summary Writer | @---summaryWriter--- |

### Agenda

<!-- If information is documented that shall not be made available for all Camundi, please use and reference an [ELT confidental note](https://drive.google.com/drive/u/1/folders/1EVfpSkZIP3yY90brsNotytYh2SPuqsbJ)-->

#### Status Check-in

- [ ] Conduct status check-in: _ensure the ELT knows status of key-items on our roadmap. Consider next target release in [product hub](https://github.com/orgs/camunda/projects/9/views/31)_.

#### Discussion Topics

_Every ELT member can add discussion points here._

- [ ] ...

#### Management Bugs and Tasks

- [ ] Review [tasks](https://github.com/orgs/camunda/projects/52) that are currently `In progress`.
- [ ] Introduce new [tasks](https://github.com/orgs/camunda/projects/52) that were put into the `Inbox`, potentially assigning them (`/autoassign`) and moving to `Backlog` or `In progress`.

----------------------

Find recordings of all meetings [here](https://docs.google.com/document/d/1PfpCZ1eo8IdVvHdaehAH68VTQLJCRYaoxbih_mQsK9c/edit).

Find ELT confidental notes or link a new note [here](https://drive.google.com/drive/u/1/folders/1EVfpSkZIP3yY90brsNotytYh2SPuqsbJ).

Find all Weeklies (incl. past ones) [here](https://github.com/camunda/engineering-leadership-team/issues?q=label%3Aweekly+).

Cf. [weekly agenda](https://github.com/camunda/engineering-leadership-team/blob/main/.github/ISSUE_TEMPLATE/WEEKLY_NOTE.md), [manually create new agenda](https://github.com/camunda/engineering-leadership-team/issues/new?template=WEEKLY_NOTE.md&labels=weekly)", "---moderator---", nextModerator), "---summaryWriter---", nextSummaryWriter)" resultVariable="asdf" />
      </bpmn:extensionElements>
      <bpmn:incoming>Flow_19rv9cw</bpmn:incoming>
      <bpmn:outgoing>Flow_10sw6vi</bpmn:outgoing>
    </bpmn:scriptTask>
    <bpmn:endEvent id="Event_1k85xel">
      <bpmn:incoming>Flow_10sw6vi</bpmn:incoming>
    </bpmn:endEvent>
    <bpmn:sequenceFlow id="Flow_10sw6vi" sourceRef="Activity_1m1fzrs" targetRef="Event_1k85xel" />
  </bpmn:process>
  <bpmndi:BPMNDiagram id="BPMNDiagram_1">
    <bpmndi:BPMNPlane id="BPMNPlane_1" bpmnElement="Process_0t4d2bh">
      <bpmndi:BPMNShape id="_BPMNShape_StartEvent_2" bpmnElement="StartEvent_1">
        <dc:Bounds x="179" y="99" width="36" height="36" />
      </bpmndi:BPMNShape>
      <bpmndi:BPMNShape id="Activity_0xtvt4q_di" bpmnElement="Activity_1m1fzrs">
        <dc:Bounds x="270" y="77" width="100" height="80" />
      </bpmndi:BPMNShape>
      <bpmndi:BPMNShape id="Event_1k85xel_di" bpmnElement="Event_1k85xel">
        <dc:Bounds x="432" y="99" width="36" height="36" />
      </bpmndi:BPMNShape>
      <bpmndi:BPMNEdge id="Flow_19rv9cw_di" bpmnElement="Flow_19rv9cw">
        <di:waypoint x="215" y="117" />
        <di:waypoint x="270" y="117" />
      </bpmndi:BPMNEdge>
      <bpmndi:BPMNEdge id="Flow_10sw6vi_di" bpmnElement="Flow_10sw6vi">
        <di:waypoint x="370" y="117" />
        <di:waypoint x="432" y="117" />
      </bpmndi:BPMNEdge>
    </bpmndi:BPMNPlane>
  </bpmndi:BPMNDiagram>
</bpmn:definitions>

Expected behavior
FEEL expressions that are valid, are not marked as invalid
Environment
Camunda Modeler 5.15.0
Also present in Web Modeler SaaS as of 18.09.
Additional context
No response

Thanks for opening this bug report. Checking the DMN spec this is likely an engine bug.
The DMN spec does not allow vertical spaces (line feeds) within a FEEL string:

I've reached out to double check this.

Confirmed to be an engine bug, even though I agree it would be a nifty feature we'd otherwise need to build with a syntax extension :slightly_smiling_face: .


Fix iOS CHIPTool commissioning

Problem
What is being fixed?  Examples:
CHIPTool on iOS doesn't use the right APIs to commission devices.
Change overview
Fix CHIPTool on iOS
Testing
How was this tested? (at least one bullet point required)

If manually tested, what platforms controller and device platforms were manually tested, and how?
Manually tested that CHIPTool iOS can pair with accessories.

Fast-track: platform-specific change by platform owner.

/rebase

/rebase


[DESIGN] Game board

🎨 Design

Game board
Implementation

Main board, chat, list of actions, score
Additional context

this prototype has too much things on screen and at the same time too much wasted space, we need to rearrange this view.

this prototype has too much things on screen and at the same time too much wasted space, we need to rearrange this view.

I repeat my reply in #2, the prototype exists to have a general idea of all the features we need to add, the design is yet to be done. DESIGN issues have already been created!


[subset] (locl) dotless i + layout features == no layout features in output file

(Please correct me if this is no issue at all, or if I describe it wrong)
If I have a font with a locl feature for dotlessi (or i.TRK, whatever you would name it) and I do a subset keeping all the glyphs and all the features for example then for the resulting file (WOFF) no features work anymore.
For example the feature is:
feature locl {
	script latn;
	language TRK exclude_dflt;
	sub [I i.smcp] by [Idotaccent idotaccent.smcp];
} locl;

If I drop the locl feature (--layout-features-=locl) there's no problem.
BTW @twardoch this also happens with TransType4.
Is the feature implemented wrong in the font, is this a WOFF problem or is this a subset problem?
It took me a couple of hours to figure out it was the "i" that was causing the problem when testing, first with a full glyph subset and features, then just 1 feature (like smcp) and a few glyphs at a time etc. jeeez, what a pain ;)

Sounds like a font issue to me.  Isn't that what your exclude_dflt mean?!

I'd need to take a look at the font files (before and after).
Also, your “locl” feature code looks alarmingly wrong. If I type in the name Yazıcıgil in all-caps (YAZICIGİL) then your font will display it as YAZİCİGİL, which of course is wrong (completely changes the pronunciation, making “yah-zee-jee-gheel” from “yah-zeh-jeh-geel”, or so).
You shouldn't replace all “I”s by “İ”s — that actually does more harm than it helps. Turkish has both I (which is a short I like in the English word “bit”) and İ (which is a long I like in “beat”).

FYI, more descussion on the Turkish "i" is at: http://typedrawers.com/discussion/932/what-is-i-latntrk


Datepicker not responding.

Hello,
There is another issue similar to this, however it was closed and I didn't want to reopen it.
Basically my datepicker is rendering, displaying hint text if I add it however when clicked on it doesn't do anything. There are no errors or anything to suggest it is broke.
I've done quite a bit of searching and only came up with adding this:
var injectTapEventPlugin = require("react-tap-event-plugin");
injectTapEventPlugin();
This did not fix the issue. Also note that everything from Material-UI is working fine this is the only component I've encountered an issue with.
Any advice would be great.
Regards
Michael

@SFPink What version of MUI are you using? What browser(s)/platform(s) are you experiencing the issue on?

MUI: 0.9.1
Using Windows 7,
Tested on Chrome 43.0, Firefox 38.0, IE 11 Doesn't work with any so I'm assuming its an issue with the project

I just built master and I'm not having any issues with tabs or the datepicker. Can you navigate to MUI in node_modules then navigate to docs and run npm install && npm start? That will build the docs site. My goal is to verify MUI behaves correctly on its own.

@SFPink Are you still experiencing issues?

Apologises for the late reply.
As I was on a deadline and couldn't get it fixed I switched to materializecss library as a temporary fix. I'm not sure whether it was something in the project breaking it but will be in touch if I move back to it.
Regards.


Docker Integration Tests Are Slow and Flaky

Summary
The integration tests powered by gnomock/docker are lowering the quality of our testing:

Each new test adds a fixed 3 second overhead to setup/teardown the docker environment.
It has become flaky, fairly regularly introducing weird socket errors that appear on random tests and go away after a retry:

--- FAIL: TestIgnoreDefaultFrozenConfigUpdate (2.97s)
    postgres_integration_test.go:47:
        	Error Trace:	postgres_integration_test.go:47
        	            				postgres_integration_test.go:673
        	Error:      	Received unexpected error:
        	            	can't perform last cleanup: read unix @->/run/docker.sock: use of closed network connection
        	Test:       	TestIgnoreDefaultFrozenConfigUpdate
        	Messages:   	Error stoping gnomock
{"level":"info","msg":"Running 13 migrations.","time":"2021-04-28T12:10:56Z"}

Scope/Requirements
Allow injecting a postgres environment in a way that will seamlessly replace gnomock when available.
Environment variables would be an clean way to do this in a way that.

Define some variables like INDEXER_TEST_POSTGRES_URL, INDEXER_TEST_POSTGRES_USER, etc
In postgres_integration_test.go : setupPostgres check for the environment variables, if available create a database on the provided server. Follow the existing interface, returning sql.DB, connection string, and a cleanup function.
In the .travis.yml file, start a postgres instance and set variables before running tests.

Urgency/Relative Priority
Medium. It's easy to workaround but will continue to become more and more of a problem over time.

I think we can just remove gnomock. Here is what others do: https://github.com/sfackler/rust-postgres. Also, it would be nice to have the docker image in the repository. It's kind of ridiculous that I have to be connected to the Internet in order to run tests.

Gnomock makes it trivial to run tests with all dependencies self contained. I'm not sure how relevant an internet connection is relevant to the discussion, the build fetches dependencies from the internet, and most make targets run a go get command that requires an internet connection.
My proposal is to support an additional option which would be used the same way as your rust-postgres example

What dependencies are you referring to?

go modules

I'm still not sure what you mean.

When you build go programs the dependencies are downloaded

But how does gnomock help?

It lets you run/debug tests without concerning yourself with any environment setup.

It basically starts a docker image. We can do the same in the makefile.

@chaihoang Did you give this to us on purpose or by mistake?

Please consult with @tolikzinovyev or me before starting work on this, we have been chipping away at this and are almost complete.


Translate error

I can't do translations after the first installation
 # deepl translate -h
deepl-translate(1) does not exist, try --help

Node : v8.14.0
deepl: v1.1.2

hey,
any solution to this?
I have the same problem.
using deepl on macos.
yarn installed with brew.
yarn: v1.16.0
node: v12.4.0
deepl: v1.1.2
any clues on how to fix this?
thanks,
camilo

This seems to be a bug in the command module which this program is based on. When looking for a subcommand, it implies that argv[1] is a subcommand name, though Node.JS reserves first two arguments for the interpreter path and for the main JS module path. I have no idea how could this happen; maybe some older versions of Node.JS reserved only one argument if a script is called via shebang, or the command module requires to cut the interpreter out manually. Will investigate it deeper later.

Same here, on linux ubuntu 18.04 with linuxbrew
yarn: v1.16.0
node: v12.5.0
deepl: v1.1.2

any work around this issue?

Same here, Manjaro 18.1, 4.19.60-1-MANJARO
yarn 1.17.3
node v11.15.0
deepl 1.1.2
Thanks for any help

+1

Same here, on linux ubuntu 18.04 with linuxbrew
yarn: v1.16.0
node: v12.5.0
deepl: v1.1.2
Switched to yarn and node installed without linuxbrew, no result.

@looksworking how can we help here?

Same here, on linux Red Hat Enterprise Linux Workstation release 7.4 (Maipo)
yarn: v1.17.3
node: v8.16.1
deepl: 1.1.2
The error msg is the same:
deepl-translate(1) does not exist, try --help
Any help or suggestion will be appreciate. Thank you in advance!

+1

I believe the DeepL started to charge their API and thus this program does not work anymore!

+1

+1 on Debian 9, deepl installed via yarn

+1 Ubuntu 19

+1 Ubuntu 19.
Has it at anytime worked? What a disappointment, this is an old bug and seems no body replies on the issues of this project

+1 Debian 10

+1 macOS

+1

+10086

CentOS Linux release 7.9.2009 (Core)

Linux centos 3.10.0-1160.42.2.el7.x86_64 #1 SMP Tue Sep 7 14:49:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

yarn version v1.22.15

Repository no longer maintained


cfep-10, feedstock support statuses

@conda-forge/core I think this is ready for/needs discussion
closes https://github.com/conda-forge/conda-forge.github.io/issues/539

I think there should be language added about the responsiveness to issues & PRs from real humans too, not just the bot

This is great! I see only two things missing:

how is standing communicated (maybe not in this PR, but perhaps worth discussing)? Is it a badge on the README? A listing on conda-forge.org?
Process for transitioning from one state to another. Time limits, state thresholds, warning & notification mechanisms, etc.

I think 1. warrants discussion, but perhaps doesn't need to be specified as part of this CFEP. 2. seems like it should be at least partly here, though, since there are a lot of questions about it, e.g. how automated can we make it? I don't see a great way to automatically threshold user interactions, but thresholding bot PRs is easy enough, though it might not be sufficient.
Questions if classification is automated:

what inputs are considered?

is "activity" relevant? e.g. some CB3 pull requests were outstanding for a very long time, but were not inactive. They were just hard!
are user-interactions (Issues, PRs) considered? I think this is really hard, but could be something like: number of issues where last activity is not from a maintainer and last comment is at least X time ago (a "known issue" label can put a hold on this to avoid forcing "still an Issue" comments). For PRs, ignore those with "WIP" in title

what are the thresholds?

are they instantaneous (i.e. 3 bot PRs open now), or time-limited (3 bot PRs open for 3 weeks)
what's the "unresponsive" threshold?
what's the time limit for unresponsive -> archived?
What sort of activity brings a repo back from unresponsive to good standing? Merging one PR? Making one comment on an Issue?

And if classification can have human input:

Is there a mechanism to propose transitions, but transitions are actually handled manually (by whom?). e.g. a user commenting "this package is broken and I'm not getting help, ping @conda-forge/unresponsive-bot"

how is standing communicated (maybe not in this PR, but perhaps worth discussing)? Is it a badge on the README? A listing on conda-forge.org?

I think this is an important implementation decision. I think we may have more than one place where this information is exposed. I definitely want it somewhere in the feedstock graph, since we could then have bots interact with it. It is also important to expose it to would be maintainers, most likely in a central place and to existing maintainers somewhere on the feedstock. I guess the would be maintainers could get something derived from the graph itself.

Process for transitioning from one state to another. Time limits, state thresholds, warning & notification mechanisms, etc.

Yep, numbers need to be assigned to things (I just filled in things that seemed not unreasonable, but I have no claim that they are right or good).

is "activity" relevant? e.g. some CB3 pull requests were outstanding for a very long time, but were not inactive. They were just hard!

The bot currently tracks the PRs it issues with a high level of fidelity, I think we can go from the most recent update (push, comment, etc.).

are they instantaneous (i.e. 3 bot PRs open now), or time-limited (3 bot PRs open for 3 weeks)

As a piece of context, the bot stop issuing version bumps after 3 have gone unmerged/closed.

What sort of activity brings a repo back from unresponsive to good standing? Merging one PR?

I think that closing/merging some of the bot PRs would be good. Since we can have three at once, it is unlikely that all need to be merged, so closing would be a low cost way to get back into good standing.

This also should include some discussion of feedstocks that have no maintainers.

We might also consider what to do when a large number of version bump PRs are rejected without cause?

@conda-forge/core
This PR falls under the CFEP policy, please vote and/or comment on this PR.
This PR needs 60% (12) of core to vote yea to pass.
To vote please leave Approve (yea) or Request Changes (nay) reviews.
If you would like changes to the current language please leave a comment or push to this branch.
This vote will end on Jan 22nd (2 weeks).

@pkgw would you please push to this branch?

@CJ-Wright Only the small-bore items that are straightforward to fix, or touching on all of my comments?

I would do all of them you think appropriate.
More broadly I think this CFEP should cover the definitions and the policies. The implementation of these hopefully doesn't need to go through the CFEP process since those are technical details and are subject to change as new approaches come about. My hope is that once this is merged there will be enough details about desired outcomes that the technical implementation (eg. is this info on the status page, is there a badge on the feedstock, is there messaging when the package is downloaded, etc.) can be discussed in a faster medium.

OK, I pushed some changes that fixed the low-level stuff and tried to give an example of splitting out a set of policies (maintainer interactions) from the strict definitions of the statuses.

merging as deferred


Publishing pipeline to production Dag

Includes work to update the alias; cache invalidation
Includes index naming work

I haven't started work on this, let's revisit it in January.


bugfix: update tqdm bar with the fixed gradient_accumulation_steps

What does this PR do?

We should use gradient_accumulation_steps to call pbar.update, instead of the current progress. tqdm will automatically calculate the progress.
Official doc link：https://github.com/tqdm/tqdm#manual

The following code can be used to validate this conclusion.
from tqdm import tqdm
import time

num_epochs = 2
for epoch in range(num_epochs):
    total_length = 16000
    pbar = tqdm(colour="blue", desc=f"Training Epoch: {epoch}", total=total_length)
    for i in range(total_length):
        if i % 100 == 0:
            print(i, pbar.total)
            # bad
            # pbar.update(i)
            pbar.update(100)
        time.sleep(0.001)
    pbar.set_description(f"Training Epoch: {epoch}/{num_epochs} completed {i}")
    epoch += 1

Feature/Issue validation/testing
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A
Logs for Test A

[ ] Test B
Logs for Test B

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Thanks for contributing 🎉!

@mreso PTAL


Upgrade from Keptn 0.7 to 0.7.1

When Keptn 0.7.1 is available, a user would like to upgrade his/her Keptn from 0.7.0 to 0.7.1.
Assumptions:

Introducing Helm for performing the upgrade is not possible because this would bring in an additional prerequisite for the user.
keptn CLI is available and CLI version 0.7.1. can be manually installed.
kubectl CLI is available.

Proposal using the Keptn CLI

Fully automated solution:

keptn upgrade

K8s cluster:            gke_sai-research_us-central1-c_keptn-prod
Namespace:              keptn
Current Keptn version:  0.7.0
New Keptn version:      0.7.1
Do you want to upgrade from 0.7.0 to 0.7.1? y/n
-------------------------------------------------------
Automatically upgrading Keptn from 0.7.0 to 0.7.1
Keptn successfully upgraded

Semi-automated solution:

keptn upgrade (--namespace=keptn)

K8s cluster:            gke_sai-research_us-central1-c_keptn-prod
Namespace:              keptn
Current Keptn version:  0.7.0
Enter the version to upgrade to: ____
Do you want to upgrade from 0.7.0 to x.y.z? y/n
-------------------------------------------------------
Upgrading Keptn from 0.7.0 to x.y.z
Keptn successfully upgraded

Info:

The current Keptn version is derived from the Helm release

TBD:

How to manage upgrade scripts?

Definition of Done:

[ ] TBD

User output:
keptn upgrade
Helm Chart used for Keptn upgrade: https://storage.googleapis.com/keptn-installer/keptn-0.7.1.tgz

Please confirm that the provided cluster information is correct:
Cluster: gke_sai-research_us-central1-c_keptn-grimmer-dev
Is this all correct? (y/n)
y
Do you want to upgrade Keptn version 0.7.0 to 0.7.1? (y/n)
y
Start upgrading Helm Chart keptn in namespace keptn
Keptn has been successfully upgraded on your cluster.


Supports use of features in App Extensions.

Features: NVActivityIndicatorViewable and NVActivityIndicatorPresenter
#119
This PR replaces the use of UIApplication.shared.keyWindow by self.data?.controller.view, thus making all the framework features available to either app or extension.
Such controller is already available since NVActivityIndicatorViewable where Self: UIViewController. vide changes.
Tests were made on NVActivityIndicatorViewExample.

@ninjaprox Do you suggest an approach to fix the tests? Apparently mine didn't work. :/

@adrianodiasx93: You can have a look at this line, the tests are still using the window to check the appearance of the UI blocker. This PR potentially solves #211 as well, and will deprecate the need of subpod NVActivityIndicatorView/AppExtension.


WRO-6793: Add delay before long Press in ui-test(QWT-2389).

Enact-DCO-1.0-Signed-off-by: Dongsu Won<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[x] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added

Need to add delay before long press in ui-test.
Need to add delay wait for loading video in ui-test(VideoPlayer).

Resolution

Add delay before long press in ui-test.
Add delay wait for loading video in ui-test(VideoPlayer).

Additional Considerations
Links
WRO-6793
Comments
It only reproduced in jenkins.

Jenkins passed: job/enact-ui-tests/3629 ~ 3633


Add back buttons

Back buttons according to discussion

I think the purpose is to keep consistent in UI while using back function. So I add back button into Buttons to emphasis on that. Adding the back icon to Icons is necessary as well.

Right, but none of the buttons are a unique 'back button', its the icon that makes it a back button, so do we need to show a separate button chapter for it?
Also, is 'back to page' a good description of the icon? maybe we could have something like 'used to navigate to previous view' or 'indicates backwards navigation when used in a button'?
is there any kind of button that cant have a back icon? if that is the case i could see a chapter explaining that and giving examples of what can be used

So I gather your point is the back buttons should not be added to 'Buttons' for incapable to serve as an individual component. This is interesting because under this circumstance we'll be requiring an agreement on keeping the consistent of back function. Is it well explained by defining a back icon?
I don't have a good idea on the description. Your suggestions are great. Let's choose one then.

closing this because of the thread on the rnd forum


Android Basics issue:

URL of codelab
https://developer.android.com/codelabs/android-basics-kotlin-write-instrumentation-tests?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fandroid-basics-kotlin-write-instrumentation-tests#3
In which task and step of the codelab can this issue be found?
"Implement the Test function section", Step 2.
Describe the problem
Codelab says "...and so the id name that is passed to the withText() function...". The correct function is withId(), not withText().
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hello @zjensen1022,
Thank you for reaching out to us and reporting this typo. It is fixed:


activeTextEditor or document.fileName loses its context

Issue Type: Bug
My extension is not working after 1.33 update.  I wrote the Language iRite extension.  The issue is with this code:
let textEditor = vscode.window.activeTextEditor;
iRiteChannel.appendLine("Building: " + textEditor.document.fileName);
The first time through the channel displays the correct path to the active file trying to compile i.e. C:\AACode\program1\src\myProgram.src.  Every time after that it displays extension-output-#2 with no path.
VS Code version: Code 1.33.0 (0dd516dd412d42323fc3464531b1c715d51c4c1a, 2019-04-04T15:14:28.026Z)
OS version: Windows_NT x64 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4310M CPU @ 2.70GHz (4 x 2693)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.91GB (1.84GB free)

Process Argv
--folder-uri file:///c%3A/AACode/vscode_irite/src

Screen Reader
no

VM
0%

Extensions (47)

Extension
Author (truncated)
Version

aessoft-class-autocomplete
AES
0.1.0

Bookmarks
ale
10.4.3

lit-html
bie
1.11.0

npm-intellisense
chr
1.3.0

path-intellisense
chr
1.4.2

jsrefactor
cms
2.18.2

bracket-pair-colorizer
Coe
1.0.61

bracket-pair-colorizer-2
Coe
0.0.28

vue-peek
dar
1.0.2

jshint
dba
0.10.20

vscode-eslint
dba
1.8.2

githistory
don
0.4.6

jquerysnippets
don
0.0.1

vscode-npm-script
eg2
0.3.5

vscode-great-icons
emm
2.1.47

prettier-vscode
esb
1.8.1

vue-inline-template
fai
1.0.1

auto-close-tag
for
0.5.6

auto-rename-tag
for
0.0.15

todo-tree
Gru
0.0.126

vscode-mocha-test-adapter
hbe
1.8.1

vscode-test-explorer
hbe
2.9.2

canvas-snippets
hol
0.3.2

vue-snippets
hol
0.1.11

beautify
Hoo
1.4.11

easy-icons
jam
0.3.1

vscode-codemetrics
kis
1.18.0

node-module-intellisense
lei
1.5.0

quickstart
lol
0.1.0

vscode-language-babel
mgm
0.0.22

code-beautifier
mic
2.2.1

easy-less
mrc
1.4.5

cpptools
ms-
0.22.1

vsliveshare
ms-
1.0.45

debugger-for-chrome
msj
4.11.3

vuejs-extension-pack
mub
1.1.3

vetur
oct
0.18.1

irite
RLW
2.5.1

vscode-javascript-booster
sbu
0.10.2

vue-vscode-extensionpack
sdr
0.2.0

vue-vscode-snippets
sdr
1.6.0

indenticator
Sir
0.6.0

es6-mocha-snippets
spo
0.2.2

vscode-status-bar-format-toggle
tom
1.5.0

vscode-jumpy
wma
0.3.1

JavaScriptSnippets
xab
1.7.2

bootstrap-v4-snippets
Zac
0.2.1

(19 theme extensions excluded)

The output channel itself is also an editor and what you are seeing is the "path" of that

Ok but my extension no longer works and it did all the way up until 1.33.  So something changed, which is fine, but can you please tell me how to fix it then?

So something changed, which is fine, but can you please tell me how to fix it then

Pretty sure that behaviour didn't change, it's like that since the first release. Not sure what your problem is and what path you expect

The problem is this:
let textEditor = vscode.window.activeTextEditor;
textEditor.document.fileName would return C:\AACode\program1\src\myProgram.src.  I would then pass that to the compile command.  vscode.window.activeTextEditor.document.fileName always returned C:\AACode\program1\src\myProgram.src.  Now it does not.  Now it will return C:\AACode\program1\src\myProgram.src one time and then it returns the extension-output-#2 or whatever.  This extension has existed in some form for a few years and this has always worked.  Anyway, how do I select the open .src file as the active file and then get the path to that active file since this no longer works?

Here is the rest of the method up to that point:
vscode.commands.registerCommand("irite.build", () => {
iRiteChannel.clear();
vscode.workspace.saveAll().then(
success => {
if (success) {
iRiteChannel.appendLine("All Files Saved");
vscode.workspace.findFiles("*.src", "system.src", 100).then(
result => {
var openTextDoc;
if (result.length > 0) {
{
if(result.length > 1){
openTextDoc = new Promise(function (resolve, reject) {
resolve(vscode.window.activeTextEditor.document);
});
              } else {
                openTextDoc = vscode.workspace.openTextDocument(result[0]);
              }

              openTextDoc.then(srcFile => {
                iRiteChannel.show();
                vscode.window.showTextDocument(srcFile).then(srcEditor => {
                  checkEngineDefined().then(engineType => {
                    checkEnginePathDefined(engineType.toString()).then(engineParameters => {
                      //got engine path and compiler from config, now init active window and push to iRiteProcessor
                      //iRiteProcessor takes argument of desired .src file path
                      let enginePath: string = vscode.workspace
                        .getConfiguration("irite")
                        .get("build.enginePath", "");
                      let compilerPath: string = vscode.workspace
                        .getConfiguration("irite")
                        .get("build.compilerPath", "");
                      let textEditor = vscode.window.activeTextEditor;
                      iRiteChannel.appendLine("Building: " + textEditor.document.fileName);

Anyway, how do I select the open .src file as the active file and then get the path to that active file since this no longer works?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/if...else

Really?  I am aware of what an if statement is.  Never mind I'll figure it out myself.    I had an extension.  It worked.  1.33 comes out.  It doesn't work.  Seems pretty straight forward and I just wanted some help figuring this out.


[FLINK-13810][Docs]Update Elasticsearch Connector label

What is the purpose of the change
For document: https://ci.apache.org/projects/flink/flink-docs-release-1.8/dev/table/connect.html#elasticsearch-connector
Sink: Streaming Append Mode
Sink: Streaming Upsert Mode
Format: JSON-only
The first lable for Elasticsearch Connector should be Source.

Documentation

Does this pull request introduce a new feature? (  no)

I believe the docs are correct as they are. ES is a sink in both cases not a source.


Update meeting room instructions

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Enable-CsMeetingRoom.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


opts.allowRequest can no longer be custom

Before commit 463d7a16a139981f1eead005b388f720be0989bd, it was possible to pass a custom allowRequest function to socket.io. That commit added the checkRequest function, and always set opts.allowRequest = this.checkRequest.bind(this);, thus overwriting the passed in value.
https://github.com/Automattic/socket.io/blob/master/lib/index.js#L225
Seems like it should be opts.allowRequest = opts.allowRequest || this.checkRequest.bind(this);.

+1

:+1:

+1


[apptools-android] Add tests to test crosswalk-app --android-targets for apptools android

Impacted tests(approved): new 14, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 14, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-6123

LGTM


Change constant name of error code 0-2, merge 4-19 to 0-2. Add error code of 'port is overridden' hint.

What is the purpose of the change
Add missing error code.
Brief changelog

Add error code of 'port is overridden' hint.
Since 4-19 is same as 0-2, this PR merged original code 4-19 to code 0-2.

Codecov Report

Merging #11181 (18742f0) into 3.1 (0b2a8f0) will decrease coverage by 4.63%.
The diff coverage is 30.00%.

@@             Coverage Diff              @@
##                3.1   #11181      +/-   ##
============================================
- Coverage     69.29%   64.66%   -4.64%
+ Complexity      127       14     -113
============================================
  Files          1502     1421      -81
  Lines         78949    59304   -19645
  Branches      13842     8720    -5122
============================================
- Hits          54711    38346   -16365
+ Misses        19484    16920    -2564
+ Partials       4754     4038     -716

Impacted Files
Coverage Δ

...t/migration/DefaultMigrationAddressComparator.java
79.54% <0.00%> (-2.28%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
66.94% <0.00%> (-6.00%)
:arrow_down:

...bo/registry/support/CacheableFailbackRegistry.java
78.82% <0.00%> (-7.69%)
:arrow_down:

...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java
82.46% <0.00%> (-4.92%)
:arrow_down:

...he/dubbo/config/context/AbstractConfigManager.java
78.57% <50.00%> (-2.52%)
:arrow_down:

.../org/apache/dubbo/common/config/Configuration.java
78.43% <100.00%> (-1.06%)
:arrow_down:

...ient/metadata/ProtocolPortsMetadataCustomizer.java
88.88% <100.00%> (-1.12%)
:arrow_down:

...luster/router/script/ScriptStateRouterFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/dubbo/test/common/impl/GreetingServiceImpl.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...zookeeper/curator/CuratorZookeeperTransporter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 497 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Reference drop-down in TWL

Issue
Currently when a user changes the chapter, verse, or reference (tsv7) field, the source side turns empty, but the target side retains the ID of the original record. I believe that we switched IDs to be unique per file a while back.  Records should be aligned on ID only, not on verse reference and ID.  This way if a GL needs to change what verse a note is in they will see the English source record even though it is in a different verse.
DoD
Changing the chapter or verse in any TSV shouldn't cause the source side to be empty. It should parallel with the original source record.
Details
It appears that the source is matched according to both the reference and the ID.  It should only be matched by the ID.  The reference needs to be removed as a keyed field.
For TWL, there is a drop-down for Reference Field,
I selected a different reference from the list. Note that the source side goes empty

@birchamp @klappy @elsylambert
This demonstration sort of implies that we should not permit updates to the "key" files in the TSV files. Same things happens in TN:

Without pouring over the code, I seem to remember that the "key" is BCV+ID (for TN); for TWL, I suppose that would be REF+ID.
Does this need to be addressed before we release 1.5?

Interesting... it does NOT create a new row. But is placed where it was added, retaining the values that were unchanged on the target side.

Then if I revert it back, it re-aligns perfectly with the source side again:

But, of course, empty rows in the target are not written out. So no harm done.

@elsylambert This does need to be fixed in future for GLs, but it isn't necessary for us to fix before the 1.5 release.  I'm moving this issue to the team backlog

translationCore Create - v1.7.0 build 165-900bcf1.
Not seeing this issue in twls on editing the Reference from the drop down. Note that the reference changes to the selected verse and updates the file with it and the same id.


adding a some hook

Summary
Adding a utility hook that takes in other functions and resolves to true if any of the functions return true. Resolves to false if all functions return falsey values. Errors thrown from a hook function are deemed as falsey and are swallowed. This is the only thing I'm not sure of. What should it do?
EDIT: It now rejects errors in the hook chain properly.
Usage is like so:
some(hook1, hook2, hook3)

Other Information
Related to this discussion https://github.com/feathersjs/feathers-permissions/issues/4
cc/ @daffl @bedeoverend

@ekryski I'd vote no on swallowing errors, at least by default. In the context of feathersjs/feathers-permissions#4 it might make sense to swallow, given that I'd imagine permissions would deny on failure, but as a generic hook, I'd prefer it to crash so I could handle unexpected errors.
Perhaps it should swallow only errors with status codes 40x? i.e. implying that the error is expected, but that might be getting a little too magical.

Ya I was thinking that if an error happens you'd probably want to know about it as it's unexpected behaviour and it should error out of the some hook chain rather than swallow it. Unless anyone has any issues with that I'm going to update it so that it doesn't treat errors as falsey and swallow them.

I dunno what code climate is complaining about but I can't access the repo in there to get more details. Just get a 422 error. Everything else is all good.

ok I'm shipping it. I need this to wrap permissions.

Sorry for not getting onto this. I'm preparing for a business supper.

np. I think it's pretty safe.

GitHub shows on the PR that Code Climate failed. You can click a link on that line to see the CC page and that eventually leads you to the issues it found.

Needs tests.

GitHub shows on the PR when Code Climate failed. You can click a link on that line to see the CC page and that eventually leads you to the issues it found.

I just get this. Must be a permissions thing.

Needs tests.

Has tests and they are passing 😄


Does not work in Safari or Internet Explorer

Buttons end up in wonky places after some javascript error. The hidden cells are not hidden.

We decided on Friday 8/21 that support for these browsers is a low priority. I'll keep this issue around, but functionality in Chrome is more important at this point.


Develop

Coverage increased (+0.08%) to 90.411% when pulling beaf3220d4c9fb68ba34ed678f2ae3df480998aa on develop into 00c5359db8e12d9eacdc1d6497432309c35dd44a on master.


[CI/CD]  Error: making Read request on Azure KeyVault Secret AZURE-COSMOS-CONNECTION-STRING from template

Already installed lasted version for azd

Output from azd version
azd version 1.1.0 (commit ea9cb12575734ee6a5f99c4d415c1a51d6f32d3e)
Describe the bug
I start adz from template following:

All working in local but when start "azd pipeline config --principal-name azd-dev-1035" actions from github I got error from github action logs:
`module.keyvault.azurerm_key_vault_secret.secrets[0]: Refreshing state... [id=https://kv-nmjmzgizmjiym.vault.azure.net/secrets/AZURE-COSMOS-CONNECTION-STRING/c2a0607621284aaf99c5f37cf5cd90gr]
Note: Objects have changed outside of Terraform
Terraform detected the following changes made outside of Terraform since the
last "terraform apply" which may have affected this plan:
module.api.azurerm_linux_web_app.web has changed
~ resource "azurerm_linux_web_app" "web" {
id                                = "/subscriptions/as1bvv2f-7461-4dfc-89c4-6b9141de7e65/resourceGroups/rg-todo-action05082023/providers/Microsoft.Web/sites/app-api-nmjmzgizmjiym"
name                              = "app-api-nmjmzgizmjiym"
tags                              = {
"azd-env-name"     = "todo-action05082023"
"azd-service-name" = "api"
}
# (18 unchanged attributes hidden)
  ~ identity {
      + identity_ids = []
        # (3 unchanged attributes hidden)
    }

    # (2 unchanged blocks hidden)
}

Unless you have made equivalent changes to your configuration, or ignored the
relevant attributes using ignore_changes, the following plan may include
actions to undo or respond to these changes.
─────────────────────────────────────────────────────────────────────────────
╷
│ Error: making Read request on Azure KeyVault Secret AZURE-COSMOS-CONNECTION-STRING: keyvault.BaseClient#GetSecret: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="Forbidden" Message="The user, group or application 'appid=as1bvv2f-bb63-483d-81a1-b16ccbefbce7;oid=as1bvv2f-2ebb-4e4c-b086-81b909046eda;iss=https://sts.windows.net/d05c6e7a-e5d1-4af0-a643-f829e6e08d31/' does not have secrets get permission on key vault 'kv-nmjmzgizmjiym;location=southeastasia'. For help resolving this issue, please see https://go.microsoft.com/fwlink/?linkid=2125287" InnerError={"code":"AccessDenied"}
│
│   with module.keyvault.azurerm_key_vault_secret.secrets[0],
│   on modules/keyvault/keyvault.tf line 65, in resource "azurerm_key_vault_secret" "secrets":
│   65: resource "azurerm_key_vault_secret" "secrets" {
│
╵
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement
Terraform planned the following actions, but then encountered a problem:
module.keyvault.azurerm_key_vault_access_policy.user[0] must be replaced
-/+ resource "azurerm_key_vault_access_policy" "user" {
- certificate_permissions = [] -> null
~ id                      = "/subscriptions/as1bvv2f-7461-4dfc-89c4-6b9141de7e65/resourceGroups/rg-todo-action05082023/providers/Microsoft.KeyVault/vaults/kv-nmjmzgizmjiym/objectId/as1bvv2f-4e37-40b4-8fac-f228fbe70700" -> (known after apply)
- key_permissions         = [] -> null
~ object_id               = (sensitive value) # forces replacement
- storage_permissions     = [] -> null
# (3 unchanged attributes hidden)
}
Plan: 1 to add, 0 to change, 1 to destroy.
ERROR: deployment failed: failing invoking action 'provision', planning deployment: planning infrastructure provisioning: terraform plan failed: err failed running terraform plan:  (exit code: 1)
TraceID: d6c2a1eea639d56e987a48f149b254e2
Error: Process completed with exit code 1.`
To Reproduce
azd up
azd pipeline config --principal-name azd-dev-1035
Expected behavior
The actions from github must without error
Environment
Information on your environment:
* Language name and version
* IDE and version : Visual Studio Code 1.81.0
Additional context
Only problem on remote github action. Local working good.
Please have a look.
Thanks
Thanh

@thanhrieker , when running/provisioning the template locally, your account (the account from az account show) is used as the principal-id.  But when running from CI/CD (github), there is a Service-Principal account (not your account).
What this means is that, after an initial local provision, the access roles for Key Vault (for example) are assigned to your account (so you can run the app locally and have access to the Key Vault). Then, later, when trying to provision again from github, terraform is detecting that the access roles will be replaced.  And there's also a Forbidden error when the ServicePrincipal on github is trying to read the KeyVault (b/c it has no access).
There are 2 solutions I can imagine:

Use the same account locally and on github.  For example, use the same ServicePrincipal locally and from GitHub.
Update your infra files to support one user account (for running locally) and a ServicePrincipal (for github). Make the infra to append roles, instead of replacing.

@vhvb1989 thanks you for your solution.
I will try one of them for sure. But I would like to report this issue for Dev team can update example to avoid this error in further.
Get back to you
Thanh

@vhvb1989 we don't run into this issue with Bicep? Should we document the above workarounds for TF?

@vhvb1989 :
I try with two if your solution and got error:
uilding AzureRM Client: Authenticating using the Azure CLI is only supported as a User (not a Service Principal). │  │ To authenticate to Azure using a Service Principal, you can use the separate 'Authenticate using a Service Principal' │ auth method - instructions for which can be found here: https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/guides/service_principal_client_secret │  │ Alternatively you can authenticate using the Azure CLI by using a User Account. │  │   with provider["registry.terraform.io/hashicorp/azurerm"], │   on provider.tf line 23, in provider "azurerm": │   23: provider "azurerm" {
Now I will try with user instance Service Principal.
Get back to you later.
Best regards
Thanh

I try to config on other way and it is working well.
Step:
I create new available on variables.tf on main:
variable "service_principal_id" {   description = "The id of service principal to deploy CI/CD"   type = string   default = "" }
Update main.tf:
`# ----------------------------
Deploy key vault
----------------------------
module "keyvault" {
source                   = "./modules/keyvault"
location                 = var.location
principal_id             = "" //var.principal_id
rg_name                  = azurerm_resource_group.rg.name
tags                     = azurerm_resource_group.rg.tags
resource_token           = local.resource_token
access_policy_object_ids = [module.api.IDENTITY_PRINCIPAL_ID, var.service_principal_id]
secrets = [ // this will saved into secrets with connection info
{
name = local.cosmos_connection_string_key //name of value
value = module.cosmos.AZURE_COSMOS_CONNECTION_STRING //keyvault will generate a unikey with version and store value of connection string there
}
]
}`
Thanks you for your help
Thanh


Issue with local_auth plugin canCheckBiometrics on Huawei Y6

Hi
There seems to be a bug where faceId is not being detected on Huawei Y6 2019.
The canCheckBiometrics function returns false for this device, not sure if it's specific to this device or if it effects others.
FYI: Pretty sure it's not my implementation as its running on a number of other Android and IOS devices perfectly.
Steps to Reproduce

Create a new flutter project and setup the local_auth library and run on a Y6.

Expected results:
The function should return true as the device has faceId enabled.
Actual results:
The function returns false.

Logs

Analyzing HunterGold...

   info • 'RaisedButton' is deprecated and shouldn't be used. Use ElevatedButton instead. See the migration guide in flutter.dev/go/material-button-migration-guide). This feature was deprecated after v1.25.0-8.1.pre. • lib/components/auth_button.dart:28:14 •
          deprecated_member_use
   info • 'textSelectionColor' is deprecated and shouldn't be used. Use TextSelectionThemeData.selectionColor instead. This feature was deprecated after v1.23.0-4.0.pre. • lib/components/register/dial_code_picker.dart:80:39 • deprecated_member_use
   info • 'textSelectionColor' is deprecated and shouldn't be used. Use TextSelectionThemeData.selectionColor instead. This feature was deprecated after v1.23.0-4.0.pre. • lib/components/register/gender_picker.dart:73:39 • deprecated_member_use
   info • 'textSelectionColor' is deprecated and shouldn't be used. Use TextSelectionThemeData.selectionColor instead. This feature was deprecated after v1.23.0-4.0.pre. • lib/components/register/nationality_picker.dart:83:39 • deprecated_member_use
   info • The value of the field '_webViewController' isn't used • lib/screens/webview_default.dart:25:21 • unused_field
   info • Name non-constant identifiers using lowerCamelCase • lib/store/middleware/api/errors.dart:30:51 • non_constant_identifier_names
   info • Avoid using braces in interpolation when not needed • lib/store/middleware/api/errors.dart:34:21 • unnecessary_brace_in_string_interps
   info • Avoid using braces in interpolation when not needed • lib/store/middleware/api/errors.dart:34:33 • unnecessary_brace_in_string_interps

[✓] Flutter (Channel stable, 2.0.3, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
    • Flutter version 2.0.3 at /Users/devonray/flutter
    • Framework revision 4d7946a68d (3 weeks ago), 2021-03-18 17:24:33 -0700
    • Engine revision 3459eb2436
    • Dart version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/devonray/Library/Android/sdk
    • Platform android-30, build-tools 29.0.2
    • ANDROID_SDK_ROOT = /Users/devonray/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

[✓] Connected device (3 available)
    • SM G935F (mobile) • 988673374a43444738 • android-arm64  • Android 8.0.0 (API 26)
    • macOS (desktop)   • macos              • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)      • chrome             • web-javascript • Google Chrome 89.0.4389.114
    ! Error: Devon’s IPhone (2) is not connected. Xcode will continue when Devon’s IPhone (2) is connected. (code -13)
    ! Error: Daisy’s iPad is not paired with your computer. To use Daisy’s iPad with Xcode, unlock it and choose to trust this computer when prompted. (code -9)

• No issues found!

Hi @devon2018
Do you have a fingerprint registered on this device? Can you please latest version of local_auth: ^1.1.2 with an official example on this device and provide complete flutter run -v logs while clicking all buttons in the example
Thanks

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


ERROR: 'NoneType' object is not iterable error during loading of training data

Any idea what would cause this? It seems like it gets through most of the examples before failing.
INFO:transformers.tokenization_utils:loading file https://s3.amazonaws.com/models.huggingface.co/bert/adamlin/NCBI_BERT_pubmed_mimic_uncased_base_transformers/tokenizer_config.json from cache at /home/ubuntu/.cache/torch/transformers/6389e7150ee74c4594a9117c0b9f0f23db49b25f47d55b7c07c8f32025238a45.1ade4e0ac224a06d83f2cb9821a6656b6b59974d6552e8c728f2657e4ba445d9
INFO:root:Writing example 0 of 19423
INFO:root:Writing example 10000 of 19423
Traceback (most recent call last):
  File "test_fast-bert.py", line 59, in <module>
    no_cache=True
  File "/home/ubuntu/anaconda3/envs/pytorch_p36/lib/python3.6/site-packages/fast_bert/data_cls.py", line 494, in __init__
    val_examples, "dev", no_cache=self.no_cache
  File "/home/ubuntu/anaconda3/envs/pytorch_p36/lib/python3.6/site-packages/fast_bert/data_cls.py", line 592, in get_dataset_from_examples
    logger=self.logger,
  File "/home/ubuntu/anaconda3/envs/pytorch_p36/lib/python3.6/site-packages/fast_bert/data_cls.py", line 138, in convert_examples_to_features
    for (ex_index, example) in enumerate(examples):
TypeError: 'NoneType' object is not iterable

This is what is running:
databunch = BertDataBunch(
    DATA_PATH,
    LABEL_PATH,
    tokenizer='adamlin/NCBI_BERT_pubmed_mimic_uncased_base_transformers',
    train_file='train.csv',
    val_file='val.csv',
    label_file='labels.csv',
    text_col='text',
    label_col=labels,
    batch_size_per_gpu=16,
    max_seq_length=512,
    multi_gpu=False,
    multi_label=True,
    model_type='bert',
    no_cache=True
)

Is there something unique about example 10000? I wonder if there are any null examples or ones with non-parsable characters?


V2 nits

Coverage decreased (-0.05%) to 92.476% when pulling 486e44b9637aeb60244b061c1d75ceec595e0455 on maydewd:v2-nits into 803215651e8e9c62484a46fc404abf57008f01a8 on PUNCH-Cyber:v2.


ECM-434 Update multiple references in single cases

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/ECM-434
Change description
Update multiple references in single cases from "plain text" to "link version"
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

I know I said yesterday during our call that this needs to be set but upon further thinking, I think whilst it is correct in some sense, there needs to be a change. I think whilst this will add the markup needed to show the url, it will still point to the old ccd case as we are still referencing the old case.


Remove additional permission limit for ci/deploy

Description
Remove additional permission limit for ci/deploy
Motivation and Context
Can't open ci/deploy/pubring.gpg for writing, Permission denied<PHONE_NUMBER>43648:error:0200100D:system library:fopen:Permission denied:../crypto/bio/bss_file.c:69:fopen('ci/deploy/pubring.gpg','wb')<PHONE_NUMBER>43648:error:2006D002:BIO routines:BIO_new_file:system lib:../crypto/bio/bss_file.c:78: Can't open ci/deploy/secring.gpg for writing, Permission denied<PHONE_NUMBER>54496:error:0200100D:system library:fopen:Permission denied:../crypto/bio/bss_file.c:69:fopen('ci/deploy/secring.gpg','wb')<PHONE_NUMBER>54496:error:2006D002:BIO routines:BIO_new_file:system lib:../crypto/bio/bss_file.c:78: gpg: can't open 'ci/deploy/pubring.gpg': Permission denied gpg: Total number processed: 0 gpg: can't open 'ci/deploy/secring.gpg': Permission denied gpg: Total number processed: 0
How Has This Been Tested?
Locally in Mac
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Major release (change is NOT backward compatible with prior release)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Looks good


[Identity] Add container instance live test

Closes: https://github.com/Azure/azure-sdk-for-python/issues/35107

API change check
API changes are not detected in this pull request.