The tabletop RPG sandbox campaign web app
apaleslimghost/apaleslimghost.github.io 2
Grab your gun and bring in the cat
Templated layout builders
apaleslimghost/async-early-error 1
Because I'm tired of typing if(err) return cb(err)
apaleslimghost/atomic-write-stream 1
stream to a file atomically
Require flippin' anything
Auto controller-view rendering
Setup and sync an S3 bucket for website serving
HTTP method filter handler combinators — npm install blvd-methods
issue openedjorgebucaran/fish-nvm
always installs latest version
no matter what version i ask for, nvm ignores it and installs 14.7. i installed it with Fundle. am i doing something wrong?
~ ⟩ nvm use 10
fetching https://nodejs.org/dist/v14.7.0/node-v14.7.0-darwin-x64
######### 7.9%^C⏎
~ ⟩ nvm use 10.22.0
fetching https://nodejs.org/dist/v14.7.0/node-v14.7.0-darwin-x64
######### 7.9%^C⏎
~ ⟩ nvm use dubnium
fetching https://nodejs.org/dist/v14.7.0/node-v14.7.0-darwin-x64
######### 7.7%^C⏎
~ ⟩ echo 10.22.0 > .nvmrc
~ ⟩ nvm
fetching https://nodejs.org/dist/v14.7.0/node-v14.7.0-darwin-x64
######### 7.7%^C
created time in 3 days
Pull request review commentFinancial-Times/n-heroku-tools
chore(scripts): update script generator and readme template
+# n-heroku-tools (nht) [](https://circleci.com/gh/Financial-Times/n-heroku-tools)
maybe rename this file so it's clearer it's a template?
comment created time in 5 days
Pull request review commentFinancial-Times/x-dash
work-in-progress: x-live-blog-wrapper
+// TODO: parsing posts will not be needed when we use v2 route on next-live-events-api+const parsePost = (event) => {+ const post = JSON.parse(event.data);++ if (!post || !post.mid || !post.textrendered) {+ return;+ }++ const textElement = document.createElement('div');+ textElement.innerHTML = post.textrendered;++ const titleElement = textElement.querySelector('p strong');+ const breakingNewsElement = textElement.querySelector('img[src*="breaking_news.gif"]');+ const isBreakingNews = Boolean(breakingNewsElement);++ if (isBreakingNews) {+ textElement.removeChild(breakingNewsElement.parentNode);+ }++ textElement.removeChild(titleElement.parentNode);++ const content = textElement.innerHTML.trim();+ const title = titleElement.textContent;+ const publishedTimestamp = (new Date(Number(post.emb) * 1000)).toISOString();++ return {+ postId: post.mid,+ author: post.authordisplayname,+ publishedTimestamp,+ isBreakingNews,+ isKeyEvent: Boolean(post.keytext),+ title,+ content+ };+};++const dispatchLiveUpdateEvent = (eventType, data) => {+ // consumer app will need to consume this event after the component is rendered. therefore,+ // we defer dispatching of this event.+ window.setTimeout(() => document.dispatchEvent(new CustomEvent(eventType, { detail: data })), 0);
please don't dispatch global events from x-dash components. if this component is used in ft-app, they way it renders article pages means up to 3 instances of this component could be on the page at once, because of preloaded swipe panes. it looks like you should be able to use x-interaction for this use case though?
comment created time in 5 days
Pull request review commentFinancial-Times/x-dash
work-in-progress: x-live-blog-wrapper
+import { h, Fragment } from '@financial-times/x-engine';+import { LiveBlogWrapper } from "./LiveBlogWrapper";+import { Serialiser, HydrationData } from '@financial-times/x-interaction';+import { listenToLiveBlogEvents } from './LiveEventListener';++const HydratedLiveBlogWrapper = (props) => {+ const serialiser = new Serialiser();++ return (+ <Fragment>+ <LiveBlogWrapper {...props} serialiser={serialiser} />+ <HydrationData serialiser={serialiser} />+ </Fragment>
i'd originally intended that using the serialiser would be left for the consuming app, since you might want to reuse the same serialiser for multiple components to reduce the output, and you might have particular requirements for where the HydrationData gets output
also i don't know if we need to think coordinating this with the various Page Kit components that output server data; x-interaction predates Page Kit and the hydration stuff hasn't been used in production yet
happy to chat further about this! 🙂
comment created time in 5 days
issue openedorogene/orogene
pain point: no control over version resolution
for projects like internal design systems that are distributed as multiple modules, being able to control which versions are used is very important. you don't want different components to be able to load e.g. different versions of the buttons component they both depend on, from a visual consistency and a CSS size point of view.
we're currently solving this by using both Bower (so we can force single installed versions) and NPM, which isn't ideal. being able to use a single tool for dependencies would simplify our workflow. this probably goes hand-in-hand with #22 because you might want different resolution strategies for different package sets
created time in 6 days
Pull request review commentFinancial-Times/n-gage
Remove deploy hashed assets deploy step
module.exports = { files: { allow: [ 'dotfiles/.stylelintrc',+ 'dotfiles/.editorconfig',
🤔 weird this wasn't needed before
comment created time in 12 days
issue openedFinancial-Times/athloi
readme does not explain "cross-dependencies" or "topological order"
- these terms are quite jargony and confusing to a newcomer
- this also means the readme doesn't adequately explain how to set up your monorepo to work with athloi (i.e. using
file:version specifiers in thepackage.json)
created time in 12 days
push eventFinancial-Times/nori
commit sha fb22e09431d41fd27839013a4b919eb11854b65e
Bump lodash from 4.17.14 to 4.17.19 Bumps [lodash](https://github.com/lodash/lodash) from 4.17.14 to 4.17.19. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](https://github.com/lodash/lodash/compare/4.17.14...4.17.19) Signed-off-by: dependabot[bot] <[email protected]>
commit sha 9ec7f7f348e2eb19e5858581c5240e6af433c3bd
Merge pull request #123 from Financial-Times/dependabot/npm_and_yarn/lodash-4.17.19 Bump lodash from 4.17.14 to 4.17.19
push time in 18 days
delete branch Financial-Times/nori
delete branch : dependabot/npm_and_yarn/lodash-4.17.19
delete time in 18 days
PR merged Financial-Times/nori
Bumps lodash from 4.17.14 to 4.17.19. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/lodash/lodash/releases">lodash's releases</a>.</em></p> <blockquote> <h2>4.17.16</h2> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/lodash/lodash/commit/d7fbc52ee0466a6d248f047b5d5c3e6d1e099056"><code>d7fbc52</code></a> Bump to v4.17.19</li> <li><a href="https://github.com/lodash/lodash/commit/2e1c0f22f425e9c013815b2cd7c2ebd51f49a8d6"><code>2e1c0f2</code></a> Add npm-package</li> <li><a href="https://github.com/lodash/lodash/commit/1b6c282299f4e0271f932b466c67f0f822aa308e"><code>1b6c282</code></a> Bump to v4.17.18</li> <li><a href="https://github.com/lodash/lodash/commit/a370ac81408de2da77a82b3c4b61a01a3b9c2fac"><code>a370ac8</code></a> Bump to v4.17.17</li> <li><a href="https://github.com/lodash/lodash/commit/1144918f3578a84fcc4986da9b806e63a6175cbb"><code>1144918</code></a> Rebuild lodash and docs</li> <li><a href="https://github.com/lodash/lodash/commit/3a3b0fd339c2109563f7e8167dc95265ed82ef3e"><code>3a3b0fd</code></a> Bump to v4.17.16</li> <li><a href="https://github.com/lodash/lodash/commit/c84fe82760fb2d3e03a63379b297a1cc1a2fce12"><code>c84fe82</code></a> fix(zipObjectDeep): prototype pollution (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4759">#4759</a>)</li> <li><a href="https://github.com/lodash/lodash/commit/e7b28ea6cb17b4ca021e7c9d66218c8c89782f32"><code>e7b28ea</code></a> Sanitize sourceURL so it cannot affect evaled code (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4518">#4518</a>)</li> <li><a href="https://github.com/lodash/lodash/commit/0cec225778d4ac26c2bac95031ecc92a94f08bbb"><code>0cec225</code></a> Fix lodash.isEqual for circular references (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4320">#4320</a>) (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4515">#4515</a>)</li> <li><a href="https://github.com/lodash/lodash/commit/94c3a8133cb4fcdb50db72b4fd14dd884b195cd5"><code>94c3a81</code></a> Document matches* shorthands for over* methods (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4510">#4510</a>) (<a href="https://github-redirect.dependabot.com/lodash/lodash/issues/4514">#4514</a>)</li> <li>Additional commits viewable in <a href="https://github.com/lodash/lodash/compare/4.17.14...4.17.19">compare view</a></li> </ul> </details> <details> <summary>Maintainer changes</summary> <p>This version was pushed to npm by <a href="https://www.npmjs.com/~mathias">mathias</a>, a new releaser for lodash since your current version.</p> </details> <br />
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
<details> <summary>Dependabot commands and options</summary> <br />
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)@dependabot use these labelswill set the current labels as the default for future PRs for this repo and language@dependabot use these reviewerswill set the current reviewers as the default for future PRs for this repo and language@dependabot use these assigneeswill set the current assignees as the default for future PRs for this repo and language@dependabot use this milestonewill set the current milestone as the default for future PRs for this repo and language
You can disable automated security fix PRs for this repo from the Security Alerts page.
</details>
pr closed time in 18 days
delete branch Financial-Times/n-tracking
delete branch : matth/page-attention-2014
delete time in 19 days
PR merged Financial-Times/n-tracking
This adds a link to the original implementation of the page attention event as used on Falcon via iJento in 2014.
https://financialtimes.slack.com/archives/CL4U9CF9B/p1594886444208500
pr closed time in 19 days
push eventFinancial-Times/n-tracking
commit sha a61b6ecba547890c33d19718830b4abf34acaf00
Update page attention documentation This adds a link to the original implementation of the page attention event as used on Falcon via iJento in 2014.
commit sha 31eceff6076385bba7deeb1d585f50bf2066e884
Merge pull request #52 from Financial-Times/matth/page-attention-2014 Update page attention documentation
push time in 19 days
pull request commentFinancial-Times/n-tracking
Update page attention documentation
i'll merge this because you're not Customer Products any more 😁
comment created time in 19 days
Pull request review commentFinancial-Times/engineering-progression
Infra examples additions from domain specific workshop
- Fixes vulnerabilities in dependencies raised by Snyk - Fixes vulnerabilities raised by the cyber security team - Doesn't introduce vulnerabilities outlined in the Open Web Application Security Project (OWASP) top 10+ - Patches EUC endpoints or servers+ - Maintains anti-virus services + - Reviews Active Directory group policies+ - Improves the Security Risk Score
feels like there's a lot of examples here and they're quite specific, can these be distilled down?
comment created time in 21 days
push eventFinancial-Times/google-amp
commit sha ffd7082d9ef2b8f9ba47be3d74f7553ec08cb5fa
use an svg icon and fallback to png png is 813 bytes and svg is 450 bytes (because we can gzip it)
commit sha dbf6cdeff7dd7ad09ec25cad7f67990e1c36883b
Update layout.html
commit sha 58192b51f28a58c88d71bfd58759cb9611280755
Merge pull request #440 from Financial-Times/JakeChampion-patch-1
push time in 21 days
delete branch Financial-Times/google-amp
delete branch : JakeChampion-patch-1
delete time in 21 days
PR merged Financial-Times/google-amp
png is 813 bytes and svg is 450 bytes (because we can gzip it)
pr closed time in 21 days
pull request commentFinancial-Times/google-amp
use an svg icon and fallback to png
ah, yep, as a Customer Products repo it should only be merged by somebody from the group. i'll do it
comment created time in 21 days
PR opened accounts-js/accounts
the users your documentation refers to are not all people that would be referred to as 'him'. using gender neutral language is welcoming to more potential users of AccountsJS
pr created time in 24 days
Fullstack authentication and accounts-management for Javascript.
fork in 24 days
created tagFinancial-Times/x-dash
:x::heavy_minus_sign::newspaper: New shared front-end components for FT.com and the FT Apps
created time in a month
push eventFinancial-Times/x-dash
commit sha f0dca6ce9ba03e0aabe038f5d58d6bd5ad21ceae
Version 1.3.0
commit sha 87f881f66f31edc52d45ec14477257fead0fa2ee
Merge pull request #504 from Financial-Times/version-1-3-0
push time in a month
PR merged Financial-Times/x-dash
Includes only changes to x-privacy-manager
pr closed time in a month
pull request commentFinancial-Times/x-dash
thanks for the PR @vbachev! since you're outside of Customer Products, by our merging guidelines we should merge this ourselves. but since this is a documentation change only, i'll do that as soon as the build passes 👍
comment created time in a month
push eventFinancial-Times/next-metrics
commit sha f95d1ffab81e60899d02a1ddc6f95babbea21cff
add note clarifying the name
commit sha 364e50773e0097449b3fbbb2692838aef9bc2ee6
Merge pull request #374 from Financial-Times/naming-things add note clarifying the name
push time in a month
PR merged Financial-Times/next-metrics
pr closed time in a month
PR opened Financial-Times/next-metrics
pr created time in a month
Pull request review commentFinancial-Times/x-dash
{ "name": "@financial-times/x-privacy-manager",- "version": "1.1.0",+ "version": "1.2.2",
you don't need to do this, it's handled by the release process. most other packages have the version set to 0.0.0.
comment created time in a month
Pull request review commentFinancial-Times/x-dash
+/**+ * Determine whether to display a prompt to the user to log in+ *+ * Specifying the loginUrl will make the Call To Action clickable+ *+ * @param {Object} [args]+ * @param {string} [args.userId] Used as a proxy for whether the user is logged in+ * @param {string} [args.loginUrl] The URL of the app's login page+ *+ * @return {string | undefined}+ */+export function getLoginPrompt({ userId, loginUrl }) {
is the intention that the consumers of the component import and call this function themselves, and pass that into PrivacyManager as a property? keeping the logic contained in the component would be clearer, but if you'd rather keep it like these, these utils should be documented
comment created time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 4120da8420f2404a16a838eab9993f980e41c7bb
add missing backslashes
push time in a month
startedapaleslimghost/minecraft-world-splicer
started time in a month
Pull request review commentFinancial-Times/n-magnet
Update n-eventpromo to fix accessibility issues
npm run jest make install make demo ```-- Demo port and host are set in config, and can be overridden via env variables.-- The demo will be served on: http://{demoHost}:{demoPort}/magnet-demo-- Example: http://local.ft.com:5005/magnet-demo++You can then visit local demonstration pages:++- [eventpromo-demo](http://localhost:5005/eventpromo-demo)+- [newsletter-demo](http://localhost:5005/newsletter-demo)
🎉
comment created time in a month
push eventapaleslimghost/pacer
commit sha 06fcf147fba004244e3c5d2689ab844185bf2826
support index routes
commit sha 612bd61ecc9d069dd4bb7a89546ab27f598a0d36
add an example which will probably eventually become an integration test i guess
push time in a month
push eventapaleslimghost/pacer
commit sha 289cfaeee76cab170e243f3f54eaa7b6fa54c77f
fancy centre aligned logo
push time in a month
push eventapaleslimghost/pacer
commit sha 58aa0aa198a7c593ae0c00d3e181b9c688d1b25f
todo list as manifesto
push time in a month
push eventapaleslimghost/pacer
commit sha be7844d6878b5661dd37ae9760e65f33d549bcd6
add router readme
push time in a month
push eventapaleslimghost/pacer
commit sha ef33340db1aaed60958b02b63c4e0d6c55d7c3a8
another imaginary feature
commit sha 0aca9373b115be7993ba37191ebf6188eef56a00
add support for slashes in route keys
commit sha c782e51de1150e81089e7061042a4d14080fd62f
simplify buildRouteTree
commit sha 5f1e1c2c8d6579b2b88a14e54a275afd8971810f
refactor lookup to allow better errors
commit sha ae45f02563260574e6679a839bd0379325364d8b
support returning multiple candidates and picking one
commit sha dba1c9ce25b260074c0744a6c470e6d3556e39d6
use holes to implement parameters
commit sha 04faddf087893fda90dfe68fb804293383f1f58e
rank candidates and pick the one with the least parameters required
push time in a month
push eventapaleslimghost/pacer
commit sha 50df1971b55e64069d244eab0f9487396c3a7f8c
imaginary features
push time in a month
push eventapaleslimghost/pacer
commit sha bc3088385cbf00201d45c44b8bc40d4dfe33ad39
add missing package lock files
push time in a month
push eventapaleslimghost/pacer
commit sha 9fab185324f5f07bd2d316b1d85f3137744737b0
add missing athloi config
push time in a month
push eventapaleslimghost/pacer
commit sha 5c1b43ccf55e8c217f8485c72a460d833f41e631
add test workflow
push time in a month
startedapaleslimghost/pacer
started time in a month
push eventapaleslimghost/pacer
commit sha 090f41f176a957a1eb5102e7ed38e501af16d37d
add router package and integration test
push time in a month
push eventapaleslimghost/pacer
commit sha 1ff9f723f3c2e3cb15ddbae751553b82c4e2a21f
rename http things, wrap effect itself
commit sha 1c480888a3d0b5f9267a3abfdc4abad201f46f44
reexport effects from main module
commit sha 25ef32107d1fb51d819bfd296a7909fc7b43b22b
add logo
commit sha 97de9ca8481cc169aa3d8ef181964e9b3635d59f
add basic top level readme
push time in a month
push eventapaleslimghost/pacer
commit sha 30e8621651e255709835ce06c6712da87e0a28af
change effect api use partial application for effectful func args instead
commit sha 5a34d894e1547914ed2eb1da96d5dac188b52213
add boneless http package
push time in a month
push eventapaleslimghost/pacer
commit sha 80a174afa2f115afaa88c9db7c51c626dd38a821
fix missing ` in readme
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha b3ce2a5216f7a8e6eb239063ab060e992dce6315
remove komatsu, it can't handle more than a few dozen spinners. more defensive
commit sha 3f8abf3e9ed246f9827af2031d25f0361623645f
1.0.3
push time in a month
created tagapaleslimghost/minecraft-world-splicer
move a section of a minecraft world into another, entities and all
created time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 06bb8978f1db0281d2708fa432ffefa812dd096c
update liquid ticks also
commit sha eb477a7e8ef307148057a67e97b9ba5c4ccf7961
1.0.2
push time in a month
created tagapaleslimghost/minecraft-world-splicer
move a section of a minecraft world into another, entities and all
created time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 9b8a50e35192f3e3e0575e11fbd69b2e13057551
fix readme example typo
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 3ee5378b70916016fbaf446cac3f9c532ff39b7e
bin
commit sha 2951a8808fc9ceefe6b8398ae903ae2dcaddd3a4
oops
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha f23ca8ff00e47ce1dff581dc5a746eeb10fb6da0
reamde
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 453baba2757b582f0f008cce660bc5bae5341531
move tile ticks
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha f67bc735d1af51c3fe7c7471e861e3ada92f304f
rewrite villagers' brains
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha c7d89e9d7ed3b458828e055b098bc2e7ce240442
correctly move entities like item frames
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 10aef1ab4c32d64a84c92ad9bd89544f998f7e3d
slightly better error handling
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha dad9d9ee6ea54f542ec35723a7879a9d2373449b
move tile entities also
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha 230eef64eea2deb7bbb01018f0a9dde637e7672e
move entities
push time in a month
push eventapaleslimghost/minecraft-world-splicer
commit sha a1238a1719ef074ac477d5e9dbbda59b7ff01977
clean up dependencies
commit sha 5a33016640c06af008c267e23de7df1dc9babb81
add komatsu spinners
push time in a month
pull request commentqvacua/vimr
vertically center-align text in linespacing, fixes #415
you can't reopen a merged PR, even if it's been removed from the base branch. i've recreated this as #805 and i'll look into that rendering issue
comment created time in a month
PR opened qvacua/vimr
this PR recreates #791
- [ ] check the rendering issue in https://github.com/qvacua/vimr/pull/791#issuecomment-646490411
pr created time in a month
push eventapaleslimghost/vimr
commit sha dd73381f923a99a4f21b89eeba872578f77bd2ab
Improve space key handling which enables <c-space> to be mapped.
commit sha 8004601b503acb8a8e417324df689be9f8959fe0
replace font popup menu with NSFontPanel
commit sha 848e5fd971e8350e7e219e1d6b48c91b567c17cb
Merge pull request #792 from apaleslimghost/font-selector replace font popup menu with NSFontPanel
commit sha 5cafc54dd0ebed3b75a1444ab8e444b9bd79b349
Merge remote-tracking branch 'cypheon/feature/space-key-handling' into develop2
commit sha 2b7980dc0946849e175cea24c31c133114e54817
Disable signing for now, activate later in the vm
commit sha 4b2023240e1aa2d8a362189c511495b387684408
Disable signing for now, activate later in the vm
commit sha 82deff6d689a0e0afb9bae17596715b7c59a2070
Bump version: snapshot/348
commit sha db0f05e58fad0e131c6fc99743c4fc68146f8bfd
Bump version: snapshot/349
commit sha fbbdb4e828b81ac54a72faba0c54d3791c404932
Bump appcast(s) to SNAPSHOT-349
commit sha af159ba051239eaf543ba8112eac097db2d90776
GH-799 Handle FS events better - Handle events only when the changed URL is present in the outline tree.
commit sha 9c4545fa7ca7516f87b64558888ff28239e94962
GH-799 Update release notes
commit sha 44e0a97868b17cd6accb7229a225ef31a16e7126
GH-799 Refactor slightly
commit sha ad526543bee39443d30733e0d21694f8e94b65b4
Update release notes
commit sha 224eb4d9881c76f3b481a0c762d6a1646ba403f8
Bump version: snapshot/350
commit sha ee202bea65ba58ca65141c8ecb928f9256caa8a5
Bump appcast(s) to SNAPSHOT-350
commit sha 59ff52ff9f105bb6a8749145e9acf6aa27e60d44
Remove function to get regular monospace font names
commit sha 228e2ef3ddeb95e9bc81063693b0d1c6e067e56d
Update deps
commit sha 0015a10c1a0f2cb9031c4683392d8c9c73f31be4
Exit when not updating appcast on develop
commit sha a5312bcab16edfe8970cf8fb77442d6d0364be3d
Bump version: v0.33.0-351
commit sha 6a99596245df8360a405afd3130267697449f8ae
Bump appcast(s) to v0.33.0-351
push time in a month
Maintain consistent spaces across components and projects in accordance to our baseline grid and design guidelines.
https://registry.origami.ft.com/components/o-spacing
fork in a month
pull request commentFinancial-Times/x-dash
Added troubleshooting Chrome error to installation instructions
this isn't an x-dash specific problem, it can happen with any FT.com app, so i think maybe it should be in the wiki instead?
comment created time in a month
pull request commentFinancial-Times/x-dash
i see this has been tested in next-article, has it been tested in ft-app yet? that's a requirement for release
comment created time in a month
issue openedFinancial-Times/nori
do not assume main branch is called "master"
this happens in a number of places, notably in the clone operation and pr undo step. don't know how to work out what the main branch is called, maybe get the info from github? or keep track of it when the repo is cloned?
created time in 2 months
push eventFinancial-Times/n-gage
commit sha 972ab0863ddf201c078ec51be84ea72462965c6b
wait for approval before starting renovate PR builds renovate PRs for common dependencies often fill up our circleci queue. a paused job waiting for approval doesn't count towards concurrent jobs, and it's straightforward to start this build if we need to test the PR on CI. because this branch name uses the same format as renovate, it should match the rule so we can test this.
commit sha 61206dd3d2e0d7eb7f6dc43ce4e727cb9095a0ce
correct filter name for renovate job
commit sha cb540fa6d049304f3a18564ae8c10220884420ad
Merge pull request #228 from Financial-Times/renovate-wait-for-approval
push time in 2 months
delete branch Financial-Times/n-gage
delete branch : renovate-wait-for-approval
delete time in 2 months
PR merged Financial-Times/n-gage
renovate PRs for common dependencies often fill up our circleci queue. a paused job waiting for approval doesn't count towards concurrent jobs, and it's straightforward to start this build if we need to test the PR on CI.
because this branch name uses the same format as renovate, it should match the rule so we can test this.
pr closed time in 2 months
pull request commentFinancial-Times/n-gage
wait for approval before starting renovate PR builds
yes m8

comment created time in 2 months
push eventFinancial-Times/n-gage
commit sha 61206dd3d2e0d7eb7f6dc43ce4e727cb9095a0ce
correct filter name for renovate job
push time in 2 months
PR opened Financial-Times/n-gage
renovate PRs for common dependencies often fill up our circleci queue. a paused job waiting for approval doesn't count towards concurrent jobs, and it's straightforward to start this build if we need to test the PR on CI.
because this branch name uses the same format as renovate, it should match the rule so we can test this.
pr created time in 2 months
create barnchFinancial-Times/n-gage
branch : renovate-wait-for-approval
created branch time in 2 months
issue commentFinancial-Times/nori
TypeError: Cannot read property 'filter' of undefined
doesn't seem to be happening on 2.0.0-beta.11, can anyone else repro it?
comment created time in 2 months
issue commentFinancial-Times/nori
TypeError: Cannot read property 'filter' of undefined
this is reproducible when running npx nori. seems to be coming from deep inside inquirer, updating that might fix it?
@andygout this is currently blocking people using nori, it'd be great to have this fixed before cutting 2.0.0 final 🙏
comment created time in 2 months
pull request commentFinancial-Times/nori
oh and by the way, since the current version is 2.0.0-beta.10 i think it's safe to release this as part of a 2.0.0. betas can contain breaking changes
comment created time in 2 months
pull request commentFinancial-Times/nori
would be great if you could prioritise #117 so there's an easier way to bulk-add repos over text files 🙏
comment created time in 2 months
issue openedFinancial-Times/engineering-progression
Add disclaimer that some links may be FT-internal
This is a public website/repo, but some links (e.g. the spreadsheet) can only be viewed by FT employees. we should include a disclaimer somewhere to make people aware of this. could be maybe an icon on the links with a tooltip? or in the footer?
created time in 2 months
issue openedFinancial-Times/engineering-progression
created time in 2 months
