profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/SimplyDanny/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Danny Mösch SimplyDanny Stuttgart, Germany You can't argue about taste, but you can about consistency.

SimplyDanny/modelica-language-vscode 14

Extension for Visual Studio Code that adds language support for Modelica

SimplyDanny/homebrew-pass-extensions 5

🔑 Homebrew formulae providing pass extensions

SimplyDanny/bitsy-llvm 4

Compiler for the Bitsy language based on LLVM

SimplyDanny/bitsy-swift 1

A small compiler, written in Swift, compiling the Bitsy language

SimplyDanny/advent-of-code 0

My solutions for Advent of Code

SimplyDanny/bitsyspec 0

Bitsy: The best language to implement when writing your first compiler or interpreter

SimplyDanny/discovery 0

Discover the world of microcontrollers through Rust!

SimplyDanny/fd 0

A simple, fast and user-friendly alternative to 'find'

SimplyDanny/git-identity 0

Manage your identity in Git

SimplyDanny/homebrew-cask 0

🍻 A CLI workflow for the administration of Mac applications distributed as binaries

issue commentmssun/passforios

ITMS-90338: Non-public API usage

It works again, doesn't it?

mssun

comment created time in 20 hours

delete branch SimplyDanny/passforios

delete branch : notification-with-otp

delete time in 20 hours

pull request commentmssun/passforios

Show notification with OTP after providing password through extension

I remember the iOS keyboard will show the 2FA code from the message. Maybe we can try to do the similar thing.

This would definitely be the best way and I also though about it, but could find anything related in the documentation. Do you have a reference? I would be happy to implement this.

SimplyDanny

comment created time in 20 hours

PR opened mssun/passforios

Show notification with OTP after providing password through extension

This simplifies the workflow for websites requiring a second factor via OTP to authenticate. There are several issues which ask for an improvement in the OTP handling, e.g. #469, #439, #180.

Showing the OTP just in a notification still requires to insert the digits manually into the appropriate fields. There is also the idea to copy the OTP automatically to the pasteboard, so that it does not need to be remembered. I'm not sure, however, whether it is a good UX if an app copies stuff to the pasteboard (overwriting its content) without the user explicitly asking for it.

+66 -68

0 comment

9 changed files

pr created time in 2 days

create barnchSimplyDanny/passforios

branch : notification-with-otp

created branch time in 2 days

issue commentmssun/passforios

Display folder name when using search

You are right. The folder paths are missing in this version. As can be seen in the screenshot above, this is definitely fixed in the current beta version (0.12.0 (33)).

comatory

comment created time in 3 days

issue commentmssun/passforios

Why download GPG key from URL?

I trust people who use a password manager based on Git, SSH and PGP to know what they are doing. Not publishing private keys is probably the first thing they have learned. 😉

leontepe

comment created time in 3 days

issue commentmssun/passforios

Display folder name when using search

Strange. It doesn't actually matter where you put the files. You always get the folder path on the right.

The feature is relatively old, but just to be sure: Which version are you using and on which device category (iPhone, iPad, ...)?

comatory

comment created time in 3 days

issue commentmssun/passforios

ITMS-90338: Non-public API usage

This is an issue in Go 1.17 and has been fixed in 1.17.1 which is available via Homebrew for half a day. So I expect the problem to be resolved with a new build.

mssun

comment created time in 3 days

PR opened mssun/passforios

Update SwiftLint to version 0.44.0
+4 -1

0 comment

2 changed files

pr created time in 6 days

create barnchSimplyDanny/passforios

branch : update-swiftlint

created branch time in 6 days

PR closed mssun/ObjectivePGP

Merge upstream

ObjectivePGP 0.17.0 has been released. We may update this fork accordingly. I am not sure whether the tags will be updated by the merge. So updating the tags might be required afterwards in addition.

+126373 -60024

0 comment

1073 changed files

SimplyDanny

pr closed time in 6 days

delete branch SimplyDanny/tealdeer

delete branch : create-cache-directory

delete time in 8 days

pull request commentdbrgn/tealdeer

Directly return in case of an empty command

The only thing that's left for this topic seems to be an explicit test highlighting code with an empty command. I added it and discarded the other changes.

SimplyDanny

comment created time in 8 days

push eventSimplyDanny/tealdeer

Danny Mösch

commit sha 7232bdd898eb42073d6b213e90b46eb1d7ad77d2

Allow SunOS override (#176) Fixes #82.

view details

Dalton

commit sha 07c715656f69d9bfd2519492685bd6f043e9b8c7

Support custom pages and patches (#142) Allow adding, overriding and extending custom pages. Fixes #98.

view details

Danny Mösch

commit sha 2a304b17f284f535a9e813c163fcc5eb84a2fc8e

Fix clippy warnings (#173)

view details

ギャラ

commit sha 42816d3d09aa59f15dbc2efe6719652c452bbedf

Fix order of derive helper attributes (#179)

view details

Dalton

commit sha 0fce79f8edc3841c17fea2c630cea4d7c0a4f4c8

Add unit tests for custom pages / patches (#183)

view details

Danny Mösch

commit sha a92e974ac26312354a2f85272eb4c32476c319f5

Update expected output for 'tar' tldr page (#186)

view details

Dalton

commit sha 388deac0794dcbaa0ceddc69d95e2208a6316466

Send all progress logging messages to stderr (#171) - Change logging messages to go to stderr - Updated tests to expect these messages in stderr instead of stdout - Changed test_markdown_rendering to no longer rely on the tldr pages repo

view details

Dalton

commit sha 4d4c7b6a5df80c955566501f16f0ca7205d82961

Fix clippy warnings (#188) * Fix clippy warnings * Update toolchain for CI workflows fmt and clippy

view details

Danilo Bargen

commit sha 3223fec5eeac65cca302ec04cdb2a63b75bfbb29

Update MSRV policy As discussed in #190.

view details

Cameron Tod

commit sha 25f736564db15b9022347c2a6eff3d7911cc6192

Docs: Fix packet/package typo (#185)

view details

Danny Mösch

commit sha b6342633cd0a0a03a85cd727dc1ff5d3cbc9dcb6

Use `which` instead of `tar` as reference tldr page in tests (#192) The page for `tar` changes pretty frequently. `which` was last updated in 2018 and is a rather simple command overall. So it is less likely that the reference in the repository has to be updated as often as the one for `tar`.

view details

Danilo Bargen

commit sha e32ce77c0c28723ba8326562ae37a10363f35b41

CI: Bump Rust version

view details

Danilo Bargen

commit sha a72ba4a7a972f34bdf71a3186e0f465965ee1c1b

Merge pull request #191 from dbrgn/msrv Update MSRV policy

view details

Sondre Nilsen

commit sha 06f771f64db2fe64d422b6a4748017bfd76a5c69

Use unbuffered IO for faster printing to stdout

view details

Sondre Nilsen

commit sha 22693c09bace8873de9f7f5afdc42eaa195cd700

Use rustls for TLS to avoid dynamic linking

view details

Sondre Nilsen

commit sha feb20d8c5c79494a827f6b3c51592dcd7df6edce

Add WriteError type, convert print_lines to return a result

view details

Danilo Bargen

commit sha 767b62d49307407428555c1bcfa5659301570441

Merge pull request #187 from sondr3/performance-fixes - Use unbuffered I/O for faster printing to stdout - Replace OpenSSL with Rustls

view details

bl-ue

commit sha a5aa822d12662b09e2533b9b9bb9d21eaaf91333

README: Update @SShrike → @severen (#193)

view details

Rithvik Vibhu

commit sha 187214e39e4102ff6288521c84dd31fefc5e2ebf

Remove AUR links for `tealdeer` and `tealdeer-bin` (#196) The package is now in the community repos.

view details

Niklas Mohrin

commit sha e7c5daa9b742b7dc9a7b640b7ddb2b0816e17c4c

Add Dockerfile for benchmarks

view details

push time in 8 days

Pull request review commentdbrgn/tealdeer

Create cache directory path if it does not exist

 impl Cache {     pub fn get_cache_dir() -> Result<(PathBuf, PathSource), TealdeerError> {         // Allow overriding the cache directory by setting the         // $TEALDEER_CACHE_DIR env variable.-        if let Ok(value) = env::var("TEALDEER_CACHE_DIR") {+        if let Ok(value) = env::var(CACHE_DIR_ENV_VAR) {             let path = PathBuf::from(value);--            if path.exists() && path.is_dir() {-                return Ok((path, PathSource::EnvVar));+            let path_exists = path.exists();+            if path_exists && !path.is_dir() {

I like the idea with the tuple. 👍

SimplyDanny

comment created time in 9 days

PullRequestReviewEvent

push eventSimplyDanny/tealdeer

Danny Moesch

commit sha eab70a98a2a70da75873c7eed4c75e620fe955ca

Ask the system only once for file metadata

view details

push time in 9 days

Pull request review commentdbrgn/tealdeer

Create cache directory path if it does not exist

 impl Cache {     pub fn get_cache_dir() -> Result<(PathBuf, PathSource), TealdeerError> {         // Allow overriding the cache directory by setting the         // $TEALDEER_CACHE_DIR env variable.-        if let Ok(value) = env::var("TEALDEER_CACHE_DIR") {+        if let Ok(value) = env::var(CACHE_DIR_ENV_VAR) {             let path = PathBuf::from(value);--            if path.exists() && path.is_dir() {-                return Ok((path, PathSource::EnvVar));+            let path_exists = path.exists();+            if path_exists && !path.is_dir() {

I did not understand the last comment correctly. Sorry for that! Now it should look like you proposed.

However, the unwrap() in line 70 is only justified because of applied short-circuit evaluation. What do you think about being a bit more explicit by writing let path_is_dir = metadata.map_or(false, |md| md.is_dir());?

SimplyDanny

comment created time in 9 days

PullRequestReviewEvent

Pull request review commentdbrgn/tealdeer

Create cache directory path if it does not exist

 impl Cache {     pub fn get_cache_dir() -> Result<(PathBuf, PathSource), TealdeerError> {         // Allow overriding the cache directory by setting the         // $TEALDEER_CACHE_DIR env variable.

It's only this one occurrence - removed it.

SimplyDanny

comment created time in 9 days

PullRequestReviewEvent

push eventSimplyDanny/tealdeer

Danny Moesch

commit sha f660f571b51de288a279c8dc37ce9d1c8dbecebb

Create cache directory path if it does not exist

view details

Danny Moesch

commit sha 8a08ac20ddbb9ac839297b4cf9a9262acdf6d4b8

Reorder conditions

view details

Danny Moesch

commit sha d7d3215f29e2f4d11cecd80854b102b61213a391

Ask the system only once for file metadata

view details

push time in 9 days

Pull request review commentdbrgn/tealdeer

Create cache directory path if it does not exist

 impl Cache {     pub fn get_cache_dir() -> Result<(PathBuf, PathSource), TealdeerError> {         // Allow overriding the cache directory by setting the         // $TEALDEER_CACHE_DIR env variable.-        if let Ok(value) = env::var("TEALDEER_CACHE_DIR") {+        if let Ok(value) = env::var(CACHE_DIR) {             let path = PathBuf::from(value); -            if path.exists() && path.is_dir() {-                return Ok((path, PathSource::EnvVar));+            if path.exists() {+                if path.is_dir() {+                    return Ok((path, PathSource::EnvVar));+                }+                return Err(CacheError(format!(+                    "Path specified by ${} is not a directory.",+                    CACHE_DIR+                )));             }-            return Err(CacheError(-                "Path specified by $TEALDEER_CACHE_DIR \-                     does not exist or is not a directory."-                    .into(),-            ));++            // Try to create the complete directory path.+            fs::create_dir_all(&path).map_err(|_| {+                CacheError(format!(+                    "Directory path specified by ${} cannot be created.",+                    CACHE_DIR+                ))+            })?;+            println!(+                "Successfully created cache directory path `{}`.",+                path.to_str().unwrap()+            );+            return Ok((path, PathSource::EnvVar));

Done.

SimplyDanny

comment created time in 13 days

PullRequestReviewEvent

push eventSimplyDanny/tealdeer

Dalton

commit sha 0fce79f8edc3841c17fea2c630cea4d7c0a4f4c8

Add unit tests for custom pages / patches (#183)

view details

Danny Mösch

commit sha a92e974ac26312354a2f85272eb4c32476c319f5

Update expected output for 'tar' tldr page (#186)

view details

Dalton

commit sha 388deac0794dcbaa0ceddc69d95e2208a6316466

Send all progress logging messages to stderr (#171) - Change logging messages to go to stderr - Updated tests to expect these messages in stderr instead of stdout - Changed test_markdown_rendering to no longer rely on the tldr pages repo

view details

Dalton

commit sha 4d4c7b6a5df80c955566501f16f0ca7205d82961

Fix clippy warnings (#188) * Fix clippy warnings * Update toolchain for CI workflows fmt and clippy

view details

Danilo Bargen

commit sha 3223fec5eeac65cca302ec04cdb2a63b75bfbb29

Update MSRV policy As discussed in #190.

view details

Cameron Tod

commit sha 25f736564db15b9022347c2a6eff3d7911cc6192

Docs: Fix packet/package typo (#185)

view details

Danny Mösch

commit sha b6342633cd0a0a03a85cd727dc1ff5d3cbc9dcb6

Use `which` instead of `tar` as reference tldr page in tests (#192) The page for `tar` changes pretty frequently. `which` was last updated in 2018 and is a rather simple command overall. So it is less likely that the reference in the repository has to be updated as often as the one for `tar`.

view details

Danilo Bargen

commit sha e32ce77c0c28723ba8326562ae37a10363f35b41

CI: Bump Rust version

view details

Danilo Bargen

commit sha a72ba4a7a972f34bdf71a3186e0f465965ee1c1b

Merge pull request #191 from dbrgn/msrv Update MSRV policy

view details

Sondre Nilsen

commit sha 06f771f64db2fe64d422b6a4748017bfd76a5c69

Use unbuffered IO for faster printing to stdout

view details

Sondre Nilsen

commit sha 22693c09bace8873de9f7f5afdc42eaa195cd700

Use rustls for TLS to avoid dynamic linking

view details

Sondre Nilsen

commit sha feb20d8c5c79494a827f6b3c51592dcd7df6edce

Add WriteError type, convert print_lines to return a result

view details

Danilo Bargen

commit sha 767b62d49307407428555c1bcfa5659301570441

Merge pull request #187 from sondr3/performance-fixes - Use unbuffered I/O for faster printing to stdout - Replace OpenSSL with Rustls

view details

bl-ue

commit sha a5aa822d12662b09e2533b9b9bb9d21eaaf91333

README: Update @SShrike → @severen (#193)

view details

Danny Moesch

commit sha a7fcf396792250ddc5816ab3be62ce3ad0c7b57f

Create cache directory path if it does not exist

view details

Danny Moesch

commit sha e96edc8f8fbea58d82904438781c56417f53547e

Reorder conditions

view details

push time in 13 days

delete branch SimplyDanny/passforios

delete branch : disable-long-press-on-directory

delete time in 18 days

delete branch SimplyDanny/passforios

delete branch : add-missing-file-headers

delete time in 18 days

delete branch SimplyDanny/passforios

delete branch : hide-keyboard-in-search

delete time in 18 days

issue commentmssun/passforios

Display folder name when using search

No. 1 is already implemented. On the right-hand side you see the folder path: Screenshot 2021-09-02 at 16 37 53 There are two password named github.com. One resides in folder, the other one in folder/it.

comatory

comment created time in 19 days